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

[Isaku Yamahata <yamahata@xxxxxxxxxxxxx>]

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