<div dir="ltr">I've just done an exercise with clang-format and tried 80, 100, 120, and 132 column formatting on the riscv arch code, see here:<div><a href="https://review.coreboot.org/#/c/coreboot/+/26512/">https://review.coreboot.org/#/c/coreboot/+/26512/</a></div><div><br></div><div>overall, after staring at it a bit, I found 132 the most readable. I find breaking strings and folding parameters to adhere to 80 column limits very irritating. I don't think that's just me.</div><div><br></div><div>if you look at the changes, it's not huge. In some cases, e.g. <a href="https://review.coreboot.org/#/c/coreboot/+/26512/2/src/arch/riscv/mcall.c">https://review.coreboot.org/#/c/coreboot/+/26512/2/src/arch/riscv/mcall.c</a>, IMHO the result is a dramatic improvement in readability and use of vertical space. </div><div><br></div><div>Note that in one place I turned formatting off. I don't think we want to make a habit of this, but thought the demo was useful.</div><div><br></div><div>ron<br><div><br></div><br></div></div>