[Ffmpeg-cvslog] CVS: CVSROOT style.sh, NONE, 1.1 commitinfo, 1.1, 1.2

Rich Felker dalias
Thu Jan 26 21:30:56 CET 2006


On Fri, Jan 13, 2006 at 11:10:28AM +0100, Nico Sabbi wrote:
> Alex Beregszaszi wrote:
> 
> >Hi,
> >
> > 
> >
> >>Modified Files:
> >>	commitinfo 
> >>Added Files:
> >>	style.sh 
> >>Log Message:
> >>Coding style script that prohibits tabs and trailing whitespace.
> >>   
> >>
> >
> >Hey, that's cool :)
> >
> >What about extending the check with other usual mistakes and adding
> >it to the mplayer tree aswell?
> >
> > 
> >
> 
> I'm against forbidding tabs in mplayer (I use only tabs)

Me too. Also we use imported or "semi-imported" code (e.g. my
pullup.c) which uses tabs and will not be happy to switch. MPlayer
rule has always been that the maintainer of the file gets to decide
the style, and in the case of my file's it's tabs-only, no-spaces.

Rich





More information about the ffmpeg-cvslog mailing list