[MPlayer-dev-eng] Trailing whitespace removal and precommit hooks

Reimar Doeffinger Reimar.Doeffinger at stud.uni-karlsruhe.de
Tue Feb 6 15:47:18 CET 2007


Hello,
On Tue, Feb 06, 2007 at 03:43:35PM +0100, Diego Biurrun wrote:
> On Mon, Feb 05, 2007 at 10:30:40PM +0100, Reimar Döffinger wrote:
> > On Mon, Feb 05, 2007 at 09:45:13PM +0100, Roberto Togni wrote:
> > [...]
> > > But something that reacts on non-mandatory cosmetics is just useless
> > > and annoying.
> > 
> > Well, if you disagree, fine. But it is _not_ useless. This kind of
> > cosmetics costs at least me real time when preparing patches for the
> > mailing list.
> > Not that much that I insist on such a change, but still.
> > But a script that only forbids real trailing whitespace would not be so
> > difficult to make I guess, would that be better?
> 
> What do you mean here by "real" trailing whitespace?

Those that have non-whitespace before them :-) (i.e. not whitespace-only
lines)



More information about the MPlayer-dev-eng mailing list