Re: [Xen-devel] [PATCH] 1 of 2: default ssid to 0

[Stefan Berger <stefanb@xxxxxxxxxx>]

xen-devel-bounces@xxxxxxxxxxxxxxxxxxx wrote on 06/25/2005 09:49:18 PM:

> 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
> > ???
> 
> It seems sensible for XenLinux code to follow the Linux CodingStyle 
document 
> (although I'm not sure all the drivers do this yet).
> 
> 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

The drivers in the Linux sparse directory, are they falling under this 
category?

> * { 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 )

If there was a script that we could run over the code, it would make life 
easier.

   Stefan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel