From mboxrd@z Thu Jan 1 00:00:00 1970 From: aq Subject: Re: [PATCH] 1 of 2: default ssid to 0 Date: Sat, 25 Jun 2005 21:18:39 -0500 Message-ID: <9cde8bff05062519186af4a17@mail.gmail.com> References: <9cde8bff0506251845587fb3b9@mail.gmail.com> <200506260249.18338.mark.williamson@cl.cam.ac.uk> 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: <200506260249.18338.mark.williamson@cl.cam.ac.uk> 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: Mark Williamson Cc: xen-devel@lists.xensource.com, Stefan Berger List-Id: xen-devel@lists.xenproject.org On 6/25/05, Mark Williamson wrote: > On Sunday 26 June 2005 02:45, aq wrote: > > looks like this patch (and also many other codes in the tree) mixed of > > tabs and spaces for indentation :-(. especially this code will be in > > mainline in the future. > > > > is it time for us to have a official *coding style guide*, and all the > > patches sent in must comply with the guide in order to be picked up > > ??? >=20 > It seems sensible for XenLinux code to follow the Linux CodingStyle docum= ent > (although I'm not sure all the drivers do this yet). >=20 > Xen-related C code that isn't destined for the Linux kernel itself has a > different coding style: > * we indent in blocks of 4 spaces and don't use tabs > * { and } are always on a new line, unlike K&R style > * we put a space between an if and the open parenthesis, and between the > parentheses and the boolean: if ( murble ) i think it is very bad idea to have 2 coding styles for code in kernel and out of kernel. because people work on all those code must always remember to adjust the editor configuration, depend on the code in or out of kernel, which is a nightmare :-( how about this simple guide? - C code complies Linux kernel coding style - Python code can use (4) spaces for indentation (as in currently)=20 regards, aq