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: https://www.flashrom.org/Development_Guidelines#Coding_style

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...

On Sun, May 6, 2012 at 7:10 AM Uwe Hermann <uwe@hermann-uwe.de> wrote:
On Sun, May 06, 2012 at 02:12:32PM +0200, Carl-Daniel Hailfinger wrote:
> Unless someone protests, I'd say 112 columns are the new absolute hard
> limit, and there is no exception for anything except tables.
> Table compression (i.e. having a few rows with unaligned elements
> following a super-long element) is good as long as the majority of
> elements in each column are aligned.
>
> Those limits only affect new code for the next 2 months starting today,
> and if we're still happy with the limit after 2 months, the rest of the
> code will be changed gradually where it helps readability.
> Should we encounter code within the next 2 months where 112 columns are
> way worse than 120, the limit will be increased to 120 with no
> additional evaluation time period.

I for one would very much prefer the 79/80 character (soft) limit per default.

No objections about breaking the rule for exception cases where this
makes sense (large tables, long lines which cannot be split nicely for
whatever reasons), but as a general guideline sticking to 79 char/line
is a very good thing IMHO.


Uwe.
--
http://hermann-uwe.de     | http://sigrok.org
http://randomprojects.org | http://unmaintained-free-software.org

_______________________________________________
flashrom mailing list
flashrom@flashrom.org
http://www.flashrom.org/mailman/listinfo/flashrom