|
| From: | Jes Sorensen |
| Subject: | Re: [Qemu-devel] [RFC][PATCH v3 11/21] virtproxy: add handler for control packet |
| Date: | Thu, 18 Nov 2010 17:22:17 +0100 |
| User-agent: | Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.12) Gecko/20101103 Fedora/1.0-0.33.b2pre.fc14 Lightning/1.0b3pre Thunderbird/3.1.6 |
On 11/18/10 17:18, Michael Roth wrote:
> On 11/18/2010 05:35 AM, Jes Sorensen wrote:
>> You should never have two closing braces in the same column like this -
>> something is wrong with the formatting.
>
> That's from using a block for the case
> switch () {
> case: {
> ...
> }
> }
>
> Alternative is to indent the "case:", which is "right" I think, but
> aligning those with switch() seems to be pretty standard to conserve space.
Why do you need braces around the case: { } ? That is not normally used
throughout the code.
Cheers,
Jes
| [Prev in Thread] | Current Thread | [Next in Thread] |