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

Diego Biurrun diego at biurrun.de
Thu Feb 1 01:22:22 CET 2007


On Thu, Feb 01, 2007 at 12:41:00AM +0100, Reimar Döffinger wrote:
> is anyone against installing a precommit hook that will disallow adding
> trailing whitespace? As a preparation to that it would also be necessary
> to remove all that are currently there in a cosmetics patch...
> I am in favour of it, and also of doing it in a way that will disallow
> whitespace-only lines (IIRC someone preferred to keep them, if that is so,
> how important is that?).

I'm all for it, but we should probably exclude external libraries.

I had planned to do something like this for configure ever since I
introduced a bug by accidentally adding a stray space to the end of one
line ...

Diego




More information about the MPlayer-dev-eng mailing list