<div dir="ltr"><div dir="ltr">This topic came up in a patch submission, and it occurred to me that we never actually wrote down canonical limits in the wiki. So I went ahead and added the proposed limits here: <a href="https://www.flashrom.org/Development_Guidelines#Coding_style">https://www.flashrom.org/Development_Guidelines#Coding_style</a></div><div dir="ltr"><br></div><div>It seems that we never really decided on 112-characters vs. 120-characters, so I wrote down 112-characters for now. At least most were satisfied with an 80-column "soft" limit and an exception to the limits for tables...</div></div><br><div class="gmail_quote"><div dir="ltr">On Sun, May 6, 2012 at 7:10 AM Uwe Hermann <<a href="mailto:uwe@hermann-uwe.de">uwe@hermann-uwe.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Sun, May 06, 2012 at 02:12:32PM +0200, Carl-Daniel Hailfinger wrote:<br>
> Unless someone protests, I'd say 112 columns are the new absolute hard<br>
> limit, and there is no exception for anything except tables.<br>
> Table compression (i.e. having a few rows with unaligned elements<br>
> following a super-long element) is good as long as the majority of<br>
> elements in each column are aligned.<br>
> <br>
> Those limits only affect new code for the next 2 months starting today,<br>
> and if we're still happy with the limit after 2 months, the rest of the<br>
> code will be changed gradually where it helps readability.<br>
> Should we encounter code within the next 2 months where 112 columns are<br>
> way worse than 120, the limit will be increased to 120 with no<br>
> additional evaluation time period.<br>
<br>
I for one would very much prefer the 79/80 character (soft) limit per default.<br>
<br>
No objections about breaking the rule for exception cases where this<br>
makes sense (large tables, long lines which cannot be split nicely for<br>
whatever reasons), but as a general guideline sticking to 79 char/line<br>
is a very good thing IMHO.<br>
<br>
<br>
Uwe.<br>
-- <br>
<a href="http://hermann-uwe.de" rel="noreferrer" target="_blank">http://hermann-uwe.de</a>     | <a href="http://sigrok.org" rel="noreferrer" target="_blank">http://sigrok.org</a><br>
<a href="http://randomprojects.org" rel="noreferrer" target="_blank">http://randomprojects.org</a> | <a href="http://unmaintained-free-software.org" rel="noreferrer" target="_blank">http://unmaintained-free-software.org</a><br>
<br>
_______________________________________________<br>
flashrom mailing list<br>
<a href="mailto:flashrom@flashrom.org" target="_blank">flashrom@flashrom.org</a><br>
<a href="http://www.flashrom.org/mailman/listinfo/flashrom" rel="noreferrer" target="_blank">http://www.flashrom.org/mailman/listinfo/flashrom</a><br>
</blockquote></div>