[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v2 7/8] xenalyze: remove traling whitespaces

On Thu, May 07, Andrew Cooper wrote:

> On 07/05/15 11:35, Olaf Hering wrote:
> > Result of "sed -i 's@[[:blank:]]\+$@@' tools/xentrace/xenalyze/xenalyze.c"
> >
> > Signed-off-by: Olaf Hering <olaf@xxxxxxxxx>
> > Cc: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
> > Cc: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>
> > Cc: Ian Campbell <ian.campbell@xxxxxxxxxx>
> > Cc: Wei Liu <wei.liu2@xxxxxxxxxx>
> Why not refresh patch 1 using git's --whitespace=fix mode?  It would
> save adding whitespace just to remove it again, and also fix up the
> other imported files.

As George said, my intention was to use the sources from hg as is.

My editor marks trailing spaces. But I have another changeset, the
result from running indent(1) on the source. It would have the same
effect.  Too bad, the result cant be used as is so I have excluded it
from this series. The manual editing of the result takes some time.


Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.