WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

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

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

There may be a few other things but I can't think of them right now ;-)

Fancy starting a Wiki page to collect coding style information?

Cheers,
Mark

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