lkml.org 
[lkml]   [2012]   [Feb]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: "whitespace coding style cleanup" broke coding style
From
Date
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.

btw: Your copy/paste lost the tab indents.

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.

It was a first pass and an overall improvement.

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

> Through which tree did this get in?

net-next

> thanks,

I'm working on improving the logging style
in isdn and will submit more patches soonish.

cheers, Joe



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