From mboxrd@z Thu Jan 1 00:00:00 1970 From: aq Subject: Re: [PATCH] xm info Date: Sun, 3 Jul 2005 21:30:46 +0900 Message-ID: <9cde8bff0507030530615998d7@mail.gmail.com> References: <9cde8bff050703001311e0e36b@mail.gmail.com> <9cde8bff05070301352470b3f9@mail.gmail.com> <9cde8bff05070301493e141ab@mail.gmail.com> Reply-To: aq Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: Content-Disposition: inline List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Keir Fraser Cc: xen-devel List-Id: xen-devel@lists.xenproject.org On 7/3/05, Keir Fraser wrote: >=20 > On 3 Jul 2005, at 09:49, aq wrote: >=20 > >> Yes: that's the only change you made in the file, except for a > >> one-line > >> change for xen_extraversion_t. > > > > i always think that code inside a switch() should be indented. that is > > a standard coding style, no? >=20 > The code is indented, of course, as it is already -- but neither Xen > nor Linux indent the braces enclosing switch or case statements, and > then indent the code one tab more. ok, so please unindent that part. regards, aq