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] remove tabs from xm/main.py

To: "aq" <aquynh@xxxxxxxxx>, "xen-devel" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: RE: [Xen-devel] [PATCH] remove tabs from xm/main.py
From: "Ian Pratt" <m+Ian.Pratt@xxxxxxxxxxxx>
Date: Wed, 18 May 2005 11:07:40 +0100
Delivery-date: Wed, 18 May 2005 10:07:13 +0000
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AcVbjM6VLqsBZmF5Tf2AwBXHmguibAABGvCA
Thread-topic: [Xen-devel] [PATCH] remove tabs from xm/main.py
> while investigating xm/main.py, i found that it is mixed with 
> tabs and whitespaces. that is annoying and may get us 
> misleaded when reading code. looks like that the conventional 
> coding style of Python (and also C?) in Xen is to use (4) 
> whitespaces instead of tabs? could anybody confirm this?
> 
> i searched in all tree and found bunch of files like this. if 
> nobody complains, i will catch them all (both C and Python 
> codes) and send in
> 1 patch to the list. any idea?
> 
> this patch replace tab with 4 whitespaces in (only) xm/main.py

Particularly for python, this kind of patch can be *really* dangerous.
Please can you confirm that you fully understand the python indentation
rules (I don't), and that the patch was generated mechanically by a
script. 

Thanks,
Ian

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

<Prev in Thread] Current Thread [Next in Thread>