[lkml]   [2002]   [Jul]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] 2.5.27 enum
    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.


    [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.
    | SuSE Labs
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2005-03-22 13:27    [W:0.065 / U:11.588 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site