|
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] |