[CRIU] [PATCH 3/3] criu-log.h: fit macros in 80 columns

Cyrill Gorcunov gorcunov at gmail.com
Thu Dec 19 23:30:27 PST 2013


On Fri, Dec 20, 2013 at 11:23:37AM +0400, Pavel Emelyanov wrote:
> >> I don't mind about this change, but frankly, Kir, 80 col terminal
> >> is definitely some oldtime limit. IIRC there were pretty long conversation
> >> on LKML about 80 col limit which (if my memory doesn't betray me) ended
> >> up that such limit should be increased up to 90-100 columns.
> > 
> > Then let's decide on a value and use it.
> 
> We're using the same coding style as the Linux kernel community does.
> It states that
> 
>   "The limit on the length of lines is 80 columns and this is a strongly
>    preferred limit.
> 
>    Statements longer than 80 columns will be broken into sensible chunks, unless
>    exceeding 80 columns significantly increases readability and does not hide
>    information."
> 
> This patch is worth applying according to this statement.

And I said "I don't mind about this change".


More information about the CRIU mailing list