Messages in this thread | | | Subject | Re: "whitespace coding style cleanup" broke coding style | From | Joe Perches <> | Date | Mon, 27 Feb 2012 08:52:29 -0800 |
| |
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
| |