From: Andrei Emeltchenko <redacted>
Date: Tue, 21 Feb 2012 22:38:16 +0200
Hi David,
On Tue, Feb 21, 2012 at 9:44 PM, David Miller [off-list ref] wrote:
quoted
From: "John W. Linville" <redacted>
Date: Tue, 21 Feb 2012 10:14:36 -0500
quoted
On Mon, Feb 20, 2012 at 07:23:24PM -0500, David Miller wrote:
quoted
From: "John W. Linville" <redacted>
Date: Mon, 20 Feb 2012 15:37:40 -0500
quoted
Here is another batch of fixes intended for 3.3. Most of the fixes
this time are for Bluetooth.
Pulled, but please read the bluetooth changes more carefully in the
future, there were a lot of coding style errors introduced this
time around.
I pinged Johan about this, and he tells me that he spoke to Marcel
as well. They were a bit unsure about the issue. Is your concern
primarily about some excessive tabbing for indentation of parameters
and such?
So you actually looked at the changes you pushed to me and you are
telling me you personally can't find anything that looks like garbage?
Really? Do I really have to point out such obvious stuff like this?
Are you serious?
Look at ca0d6c7ece0e78268cd7c5c378d6b1b610625085 ("Bluetooth: Add
missing QUIRK_NO_RESET test to hci_dev_do_close")
You tell me what the heck you think of this thing.
- if (!test_bit(HCI_RAW, &hdev->flags)) {
+ if (!test_bit(HCI_RAW, &hdev->flags) &&
+ test_bit(HCI_QUIRK_NO_RESET, &hdev->quirks)) {
That's disgusting, it jumps off the screen and says "Hello, I am ugly
as sin". How in the world can you miss something like this? Four
TABs on the second line? Why? I can't believe we even have to discuss
something like this, seriously.
Sorry did we understand wrong text of Linux coding style? Or do we need
right interpretation of it?
I'm not engaging in this conversation any more, I don't care what CodingStyle
says, the quoted code is garbage.