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

Ivan Kalvachev ikalvachev at gmail.com
Thu Feb 1 01:29:10 CET 2007


2007/2/1, Reimar Döffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de>:
> Hello,
> 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?).

What do you mean by whitespace-only lines? Would you remove the lines
or only the whitespaces in them?

I don't mind removing whitespaces from empty lines.

Also, would that be in power only for .[ch] files or for all other
text files (Makefile,man,xml)?



More information about the MPlayer-dev-eng mailing list