Search Linux Wireless

Re: [PATCH] Bluetooth: Fix coding style

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, May 6, 2012 at 3:53 PM, David Miller <davem@xxxxxxxxxxxxx> wrote:
> From: David Herrmann <dh.herrmann@xxxxxxxxxxxxxx>
> Date: Sun, 6 May 2012 19:46:46 +0200
>
>> 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
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Wireless Personal Area Network]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Hiking]     [MIPS Linux]     [ARM Linux]     [Linux RAID]

  Powered by Linux