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
signature.asc
Description: OpenPGP digital signature
_______________________________________________ Mailing list: https://launchpad.net/~launchpad-dev Post to : launchpad-dev@lists.launchpad.net Unsubscribe : https://launchpad.net/~launchpad-dev More help : https://help.launchpad.net/ListHelp
