Re: [PATCH RFC 4/4] xen-netfront: Add support for multiple queues

[David Vrabel <david.vrabel@citrix.com>]

On 16/01/14 10:41, Andrew Bennieston wrote:
> On 16/01/14 10:39, David Vrabel wrote:
>> On 16/01/14 10:24, Andrew Bennieston wrote:
>>> On 16/01/14 00:27, Wei Liu wrote:
>>>> On Wed, Jan 15, 2014 at 04:23:24PM +0000, Andrew J. Bennieston wrote:
>>>>
>>>>> +            goto error;
>>>>> +        }
>>>>> +        snprintf(path, pathsize, "%s/queue-%u",
>>>>> +                dev->nodename, queue->number);
>>>>> +    }
>>>>> +    else
>>>>> +        path = (char *)dev->nodename;
>>>>
>>>> Coding style. Should be surounded by {};
>>>
>>> OK.
>>
>> Linux style is single line blocks are not surrounded by braces.  You
>> should have the else on the same line as the preceeding } though.
>>
>> i.e.,
>>
>> if (...) {
>>     one_line()
>>     two_line()
>>     red_line()
>>     blue_line()
>> } else
>>     a_line()
>>
>> David
>>
> Right; I'll make sure this consistently done throughout.

Nope. Turns out I'm wrong.

In my defense, the last time I read CodingStyle (which was a
considerable time ago!) it didn't say this.

David