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

Uoti Urpala uoti.urpala at pp1.inet.fi
Tue Feb 6 16:14:51 CET 2007


On Tue, 2007-02-06 at 16:03 +0100, Guillaume POIRIER wrote:
> I'm a bit late to comment on the subject, but while I like the idea of
> not having trailing spaces snucking in, based on my experience with
> ffmpeg's pre-commit script, it's really a pain sometimes when I apply
> a 3rd party patch that contains such spaces, especially when I'm not
> using my favourite text editor that can automatically "fix" this
> problem.
> 
> Well, you could object that I just should stick to my editor all the
> time... trouble is, it's not always available on the machine I'm using
> to commit, so I have to switch to manual.

If the machine has enough programs installed to test compile/run before
committing then it should have enough to make stripping trailing
whitespace easy.




More information about the MPlayer-dev-eng mailing list