[MPlayer-DOCS] CVS: main/DOCS/man/en mplayer.1,1.749,1.750

[Guillaume POIRIER]

Hi,
Le lun 27/09/2004 à 21:46, Diego Biurrun a écrit :
> Guillaume Poirier CVS writes:
> > 
> > Modified Files:
> > 	mplayer.1 
> > Log Message:
> > Sws default setting correction, and random wordings
> 
> Seems like a bit more slipped through..
> 
> > -.B iframe=<value>
> > +.B keyint=<value>
> 
> Isn't this (and much of what comes below) from the XviD patch Loren
> Merritt sent to dev-eng? 

It's for x264 actually.


> I have a hunch you just had it in your local
> tree and committed by mistake.  Please doublecheck your changes with
> 'cvs diff' before committing.  It also helps to have a development
> tree checked out readonly that you can mess with and test patches in
> and a writable clean commit tree that only gets the things you want to
> commit.  That's how I work.

That's how I work usually also, just as written on
DOCS/tech/cvs-howto.txt, but since Murphy's laws seems to affect me a
whole too much today, I happened to commit my development tree.
I really feel horrible. I guess I really, really should do a cvs diff
before committing anything, even with my 36k internet access in order
not to be embarrassed like tonight.

Thanks Diego for not raising your tone about such a dumb mistake, I
guess being embarrassed like I've been will teach me a lesson. :-)

Now it's time for me to have a rest, I'm not doing anything good today.
:-(

Guillaume