[MPlayer-cvslog] r19188 - trunk/mplayer.c

[Diego Biurrun]

On Sun, Aug 13, 2006 at 11:00:38PM +0200, Michael Niedermayer wrote:
> 
> On Sun, Aug 13, 2006 at 03:08:27PM +0200, Roberto Togni wrote:
> > 
> > - pre commit scripts are imo very annoying, especially if they don't
> > point out exactly what is wrong and where the problem is; 
> 
> fully agree, the script must print description & line number

I'll look into improving this.

> > But there's a thing that worries me, not related only to this specific
> > subject: MPlayer was developed for years with few, simple rules; why
> > now we have the need to regulate every single detail of development?
> 
> hmm, the way i saw this precommit check was more as a tool to prevent
> unintentional adding tabs or such not to somehow restrict what mplayer
> developers really would want to conciously do

Yes.  They are meant to protect devs from simple and stupid mistakes.

Diego