[Buildroot] [PATCH 4/5] support/testing: fix remaining code style

[Peter Korsgaard <peter@korsgaard.com>]

>>>>> "Arnout" == Arnout Vandecappelle <arnout@mind.be> writes:

Hi,

 >  For me, the ideal is to wrap at 80 columns most of the time, but allow
 > exceptions up to something like 105 columns. 105 is still reasonable for
 > side-by-side on a wide display with some room for scrollbars. I know, Yann, that
 > you can easily fit 132 characters on side-by-side, but you've got an eagle-eyed
 > font size :-) That said, I can live with 132 as the absolute maximum. However, I
 > do think we should _normally_ wrap at 80 columns.

 >  Standardizing on 80 columns is good because it's what we do in the rest of
 > Buildroot. Allowing exceptions is good because there are some lines you really
 > don't want to split (strings, mostly).

 >  So ideally, I'd like to have a warning for anything about 80 columns, and an
 > error for anything over 105 (or 132). Unfortunately, I don't think flake8 allows
 > that. So let's say we put 132 in the config file, but enforce 80 columns in review.

Agreed! I also think sticking to 80 columns where possible would be good.

-- 
Bye, Peter Korsgaard