[MPlayer-dev-eng] [PATCH] changing playback speed (live Fast Forward/Slow Motion)

Diego Biurrun diego at biurrun.de
Sun Aug 29 18:13:49 CEST 2004


Taros666 writes:
> 
> > .IPs "INS and DEL"
> >-next/\:previous alternative source (ASX playlist only)
> >+next/\:previous alternative source (ASX playlist only).
> >
> >WTH? Why do you change this?
> >
> If you look at the rest of the lines, all lines have fullstops at the 
> end. Only these doesn't. So I changed it...
> Maybe on of the develops should just remove ALL fullstops or put 
> fullstops at EVERY line. I just didn't like the idea of having fullstops 
> sometimes, and sometimes not. But I am not a native speaker...

Nope.  Only full sentences have periods at the end.  I'm open to these
kinds of changes, but in this case that part of the man page has
already been reviewed and the punctuation is intended.

Anyway, this is completely unrelated to your patch about changing
playback speed and should thus not be included in it.  If you want to
see this changed, send a separate patch

> btw. I forgot that there are 3 new commands for input.conf / slave commands
> speed_set [float]    set the speef to a specific value
> speed_incr [float]   add [float] to the current playback speed
> speed_set [float]    multiply the current speed with [float]   

These would need to be documented in DOCS/tech/slave.txt.

Patch rejected, please fix up the documentation.

Diego




More information about the MPlayer-dev-eng mailing list