lkml.org 
[lkml]   [2012]   [Feb]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: "whitespace coding style cleanup" broke coding style
    On 02/27/2012 05:52 PM, Joe Perches wrote:
    > On Mon, 2012-02-27 at 09:59 +0100, Jiri Slaby wrote:
    >> commit 475be4d85a274d0961593db41cf85689db1d583c
    >> Author: Joe Perches<joe@perches.com>
    >> Date: Sun Feb 19 19:52:38 2012 -0800
    >>
    >> isdn: whitespace coding style cleanup
    >>
    >> isdn source code uses a not-current coding style.
    >> ...
    >> Done with emacs and some scripts and some typing.
    >> ...
    >> --- a/drivers/isdn/gigaset/interface.c
    >> +++ b/drivers/isdn/gigaset/interface.c
    >> ...
    >> @@ -680,9 +680,9 @@ void gigaset_if_initdriver(struct gigaset_driver
    >> *drv, const
    >> char *procname,
    >> goto enomem;
    >>
    >> tty->magic = TTY_DRIVER_MAGIC,
    >> - tty->type = TTY_DRIVER_TYPE_SERIAL,
    >> - tty->subtype = SERIAL_TYPE_NORMAL,
    >> - tty->flags = TTY_DRIVER_REAL_RAW |
    >> TTY_DRIVER_DYNAMIC_DEV;
    >> + tty->type = TTY_DRIVER_TYPE_SERIAL,
    >> + tty->subtype = SERIAL_TYPE_NORMAL,
    >> + tty->flags = TTY_DRIVER_REAL_RAW |
    >> TTY_DRIVER_DYNAMIC_DEV;
    >>
    >> Hmm, please, fix your scripts or emacs!
    >
    > Hi Jiri.
    >
    > I'm not fixing emacs.
    >
    > Notice that the code improperly uses commas and
    > not semicolons for line terminations like the
    > lines immediately above and below it.

    Oh, and you see that you made it actually _worse_? The patch should not
    touch the code at all. It is obviously totally broken. In a separate
    patch you might do s@,@;@ instead.

    > btw: Your copy/paste lost the tab indents.

    Yes, but as I did not expect anybody to apply it, I see no problem in that.

    > The emacs pass was not perfect, nor was it
    > intended to be perfect. There are _way_
    > too many coding style defects in isdn for
    > that.

    Hmm, but did not we conclude some time ago that we will not touch code
    just to perform a whitespace cleanup?

    > It was a first pass and an overall improvement.

    I hope no other passes are going to happen there or anywhere in TTY
    drivers. I really do not want to solve zillion collisions in my ~100
    local patches due to whitespace changes, sorry.

    That was exactly the reason why not to cleanup the files just to be
    checkpatch-proof.

    > "It's quite a bit more like what passes
    > for the current kernel coding style though."

    Yes, but it does not pass our brain, does it? One should throw
    "checkpatch --file" or alike away, finally.

    thanks,
    --
    js
    suse labs


    \
     
     \ /
      Last update: 2012-02-27 18:15    [W:2.350 / U:0.084 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site