Re: [PATCH] 2.5.27 enum

[Dave Jones <davej@suse.de>]

On Mon, Jul 22, 2002 at 04:01:18PM -0400, Benjamin LaHaise wrote:
 > On Mon, Jul 22, 2002 at 12:53:21PM +0200, Marcin Dalecki wrote:
 > > - Fix a bunch of places where there are trailing "," at the
 > >    end of enum declarations.
 > 
 > Please don't apply this.  By leaving the trailing "," on enums, additional 
 > values can be added by merely inserting an additional + line in a patch, 
 > otherwise there are excess conflicts when multiple patches add values to 
 > the enum.

Gratuitous 'cleanups' with no real redeeming feature also have another
downside which a lot of people seem to overlook.  They completely screws
over anyone who also has a pending patch in that area if Linus applies it.

For most people this is five minutes work as they fix up by hand
the single reject in one or two places.  For people like myself keeping
a large patchset, this is a lot of extra work for absolutely no gain.
Two kernels later, someone adds a new sysctl which re-adds the , at
the end anyway.

We have much bigger problems to fix than silly[1] things like this.

        Dave

[1] Maybe silly is the wrong word to use, but I didn't want to use
    'trivial' for fear of putting down the usefulness of Rusty's
    trivial patches.

-- 
| Dave Jones.        http://www.codemonkey.org.uk
| SuSE Labs