On Wed, May 04, 2011 at 10:44:14AM -0700, Mike Stump wrote: > On May 3, 2011, at 10:27 PM, Michael Witten wrote: > > To what do we owe this tradition other than laziness? > > By flat out rejecting style fixing patches, you preserve the > annotations made by vc-blame (svn blame). That's the only reason that > I am aware of. Laziness can't be the reason to reject the hard work > of someone that wants to clean up the code. > > Like others, I don't find that compelling enough personally.
I think accepting such style patches for documentation for something like this carries less downsides that doing the same for code. The reason is that I assume that documentation tends to be added in bulk, so if you see blame output that looks like: r178 ahacker r178 ahacker r178 ahacker r509372 bhacker r178 ahacker r178 ahacker r178 ahacker it's pretty obvious what's going on. :) Whereas seeing similar things in code could hide quite a bit. Deleting end-of-line whitespace in docs seems pretty safe. Re-formatting complete paragraphs of documentation possibly less so. -Nathan
