Thread (22 messages) read the whole thread 22 messages, 8 authors, 2012-05-07

Re: [PATCH] Bluetooth: Fix coding style

From: Lucas De Marchi <hidden>
Date: 2012-05-07 19:40:01
Also in: linux-wireless

On Sun, May 6, 2012 at 3:53 PM, David Miller [off-list ref] wrote:
From: David Herrmann <redacted>
Date: Sun, 6 May 2012 19:46:46 +0200
quoted
Whose rules are they?
Find me an example in another major core subsystem, let's use
The ones he pointed out, coming from Linus aren't enough?

mm/memory.c as an example as that file gets hit by a lot of people,
that uses the multi-line conditional TAB-only crap you guys seem to
keep using.
Look again at this file and you'll see there is mixed style.

They don't.  All the examples you'll find are of the form:

       if (a &&
           b)

not:

       if (a &&
                       b)

like I see happening in bluetooth all the time.

How did you think they figured that out?  Did they read someone's
mind?
No, they used the default in emacs or another editor that has this as
default, i.e. mix tabs and spaces


Regards,
Lucas De Marchi
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help