[Ffmpeg-devel] [PATCH]: A/V sync on nsv streams.

elupus elupus
Sat Oct 28 23:00:20 CEST 2006


> >>
> >> Patch applied (without the trailing spaces that drive me crazy)
> >>
> >> Guillaume
> >>
> > Can't we commit a script that fixes trailing spaces and 
> tabs to the svn
> > server ?
> 
> No.  People should learn to write proper code.  It's really not that
> hard.
> 
> -- 
> M?ns Rullg?rd
> mru at inprovide.com


Okey, since I started this sorry about the trailing blanks. Auto indentation
feature of the editor I'm using has a bad habit of causing them. Didn't
check the patch closely enough before submitting. Like m?ns pointed out, I
should learn to code (or mainly check my patches before submitting them).

An auto feature on commit would be very bad. Changing code on the way into
cvs causes the commiters code to be different from what ends up in cvs, wich
imho is very bad.

Just my 2 cents.

/Joakim





More information about the ffmpeg-devel mailing list