launchpad-dev team mailing list archive

[curtis Hovey <curtis.hovey@xxxxxxxxxxxxx>]

On 01/30/2012 02:15 PM, Richard Harding wrote:
> On Mon, 30 Jan 2012, curtis Hovey wrote:
> 
>> I removed the 78-character max line length exception from pocket-lint.
>> Line lengths are checked against 80-character lengths which is the most
>> common rule for projects I hack on. We can choose to change to the
>> standard, or I can land a change to Lp's wrapper to call pocket-lint
>> with the preferred line length:
>>     pocketlint --max-length=78
>>
>> What say you? Do you want to keep the current Lp exception for
>> 78-character max line length?
> 
> My experience has always been 79, so 78 had me adjusting my config, but
> close enough. 80 works just as well for me, as long as we don't go all 120
> or whatever people with wide displays and fullscreen editors like these
> days. :)

79 or 80 is the same. The subtlety depends on the indexing of the linter
being wrapped. pep8 counts from 0 so it uses 79 to check 80. Other start
from 1, thus use 80 as the max. pocketlint uses 80 and reconciles the
number as needed.

-- 
Curtis Hovey
http://launchpad.net/~sinzui

Attachment: signature.asc
Description: OpenPGP digital signature