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-ia64-devel

[Xen-ia64-devel] Re: [patch] lindent dom_fw_foo code

To: Jes Sorensen <jes@xxxxxxx>
Subject: [Xen-ia64-devel] Re: [patch] lindent dom_fw_foo code
From: Isaku Yamahata <yamahata@xxxxxxxxxxxxx>
Date: Fri, 25 May 2007 18:09:24 +0900
Cc: Alex Williamson <alex.williamson@xxxxxx>, xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Fri, 25 May 2007 02:07:39 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <4656A476.1060506@xxxxxxx>
List-help: <mailto:xen-ia64-devel-request@lists.xensource.com?subject=help>
List-id: Discussion of the ia64 port of Xen <xen-ia64-devel.lists.xensource.com>
List-post: <mailto:xen-ia64-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=unsubscribe>
References: <4655B354.7020004@xxxxxxx> <1180045207.6134.366.camel@bling> <20070525014114.GA2128%yamahata@xxxxxxxxxxxxx> <4656A476.1060506@xxxxxxx>
Sender: xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.4.2.1i
On Fri, May 25, 2007 at 10:55:18AM +0200, Jes Sorensen wrote:
> Isaku Yamahata wrote:
> > On Thu, May 24, 2007 at 04:20:07PM -0600, Alex Williamson wrote:
> > 
> >> this one that convert to Linux style.  Are there other opinions on this
> >> before we set a precedent?  Thanks,
> > 
> > I don't insist on any particular styles.
> > If linux style under ia64 directory is the consensus,
> > I will follow it.
> > 
> > If the patch is applied, we may want to modify the following line too.
> > * indent-tabs-mode: nil
> 
> Sorry, I don't understand your reasoning for this, this would
> deliberately make Emacs use spaces instead of tabs. The idea is to use
> tabs as much as possible. So if anything indent-tabs-mode should be set
> to true IMHO. Otherwise we just go back and increase the file size by
> 15%.

Probably I should have be clearer.
Your patch left the line "* indent-tabs-mode: nil" unmodified.
So I wanted to suggest 
- * indent-tabs-mode: nil
+ * indent-tabs-mode: t

-- 
yamahata

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