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

[Diego Biurrun]

On Tue, Feb 06, 2007 at 04:03:38PM +0100, Guillaume POIRIER wrote:
> 
> On 2/1/07, Reimar Döffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de> 
> wrote:
> >is anyone against installing a precommit hook that will disallow adding
> >trailing whitespace?
> 
> 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.

Have you tried the clean-diff script from the root of the FFmpeg source
tree?

Diego