Re: [PATCH] libsvn_wc #1, result_pool/scratch_pool

Martin Hauner wrote:
> I also wonder if I should modify the formatting where the lines get longer
> than 72 characters. There are a lot of other lines that don't strictly follow

Quoting HACKING:
[[[
Restrict lines to 79 columns, so that code will display well in a minimal
standard display window. (There can be exceptions, such as when declaring a
block of 80-column text with a few extra columns taken up by indentation,
quotes, etc., if splitting each line in two would be unreasonably messy.)
]]]

IMHO 76 is nicer, so code can still be indented without redoing the
wrapping. It's not a showstopper, really, if a line gets longer, but it's
nice to adhere to the limit nevertheless.

Martin, just a quick rant: if you do cosmetic changes, it's nice to
- not mix cosmetic changes with functional changes in a patch, and
- group a whole bunch of cosmetic changes in a single patch.

Anyway, if you don't have commit access, I guess it's fine to not bother too
much about cosmetic fixes.

~Neels

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2386519