[MPlayer-dev-eng] [RFC] Removing all trailing whitespace

Ivan Kalvachev ikalvachev at gmail.com
Mon Aug 21 01:38:48 CEST 2006


2006/8/21, Diego Biurrun <diego at biurrun.de>:
> On Sun, Aug 20, 2006 at 11:59:48PM +0200, Michael Niedermayer wrote:
> >
> > On Sun, Aug 20, 2006 at 06:49:30PM +0200, Ivo wrote:
> > >
> > > How about running the following on the whole source tree:
> > >
> > > find . -name '*.[ch]' -exec sed -i 's/[ \t]*$//' {} \;
> > >
> > > and then:
> > >
> > > svn ci
> >
> > :)
> >
> > iam in favor of it, though i suggest you do a quick svn di before ...
>
> I'm also in favor, though I'd suggest doublechecking with
>
>   svn diff --diff-cmd diff --extensions -b
>
> to make sure no non-whitespace changes crept in.


If we are going to make cosmetic changes, why don't make and proper
indent. At least for mplayer.c and mencoder.c

I'm also wondering about converting tabs to spaces (only .h and .c files ;)

All these whitespace changes could be ingored when needed to apply
patch by using --ignore-whitespace/-l.



More information about the MPlayer-dev-eng mailing list