[Ffmpeg-devel] [PATCH] from DivX, Part 1: cosmectic changes

Diego Biurrun diego
Thu Dec 22 19:13:47 CET 2005

On Thu, Dec 22, 2005 at 12:12:11PM -0500, The Wanderer wrote:
> On 12/22/2005 09:34 AM, Diego Biurrun wrote:
> >On Thu, Dec 22, 2005 at 09:14:35AM +0100, Guillaume POIRIER wrote:
> >
> >>On 12/22/05, Diego Biurrun <diego at biurrun.de> wrote:
> >>
> >>>OK, I've replaced all tabs with spaces along with some
> >>>prettyprinting on the code where it was necessary after the tab
> >>>removal, in related places and a few random places along the way.
> >>
> >>How did you do it? Has it been pretty much been like running:
> >>expand | sed 's/ *$//' over the files, and removing all trailing
> >>spaces?
> >
> >I just repeated
> >
> >  find . -exec sed -i -e 's/ *$//' -e 's/	*$//' {} \;
> >
> >a few times to remove all trailing whitespace.
> Hmm. How does that fail to remove whitespace on lines which consist only
> of whitespace?

It does not fail; by design, not by accident.

> I do not consider those to be "trailing whitespace";

Then reconsider, since it falls under the definition of trailing
whitespce for pretty much the rest of the world.  Every whitespace
character after the last non-whitespace character on a line is trailing
whitespace.  If the number of non-whitespace characters on a line
happens to be zero, then all the whitespace on that line is trailing.

> they exist for convenience of indentation, and I'd be bothered by
> trying to work with code which did not have them there.



More information about the ffmpeg-devel mailing list