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/
Home Products Support Community News


Re: [Xen-devel] [patch] libxc whitespace

To: Hollis Blanchard <hollisb@xxxxxxxxxx>
Subject: Re: [Xen-devel] [patch] libxc whitespace
From: Muli Ben-Yehuda <mulix@xxxxxxxxx>
Date: Sat, 15 Apr 2006 10:47:53 +0300
Cc: xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Sat, 15 Apr 2006 00:49:08 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <1145045311.8498.13.camel@xxxxxxxxxxxxxxxxxxxxx>
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>
References: <1145045311.8498.13.camel@xxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.5.11+cvs20060126
On Fri, Apr 14, 2006 at 03:08:31PM -0500, Hollis Blanchard wrote:
> I noticed there is an prodigious amount of trailing whitespace in
> tools/libxc. This patch, a result of running
>       perl -p -i -e 's/\s+$/\n/g' tools/libxc/*.[ch]
> removes it all. You could also run the perl yourself to produce the same
> effect.
> There were also a few tabs that had snuck in. I left the headers alone,
> but fixed the instances in code.
> As usual, the IA64 code contains some truly bizarre use of whitespace,
> but I resisted the temptation.
> The resulting patch is ~140KB, so I've gzipped it. It introduces no
> functional changes, and has been compile-tested on x86_32. Please

Yes please!!!

Could we just run this on the entire Xen and tools trees?

Muli Ben-Yehuda
http://www.mulix.org | http://mulix.livejournal.com/

Xen-devel mailing list

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