[MPlayer-dev-eng] [PATCH] makes the seek sizes configurable (arrows/pageup/down)

[Anders Johansson]

Hi,

I think the patch is a great idea (need it myself) but I think the
interface should follow mplayers extended option interface, i.e.

mplayer -seek s=10:m=300:l=1000

IMHO one could also put the vars seek_to_XXX in a small struct to
reduce the number of global variables, this especially if Danny will
go ahead with his keybord idea.

I would like to apply it if above changes are made, comments?

Cheers,

//Anders

> How should I interpret the lack of response?
> Is this patch wrong in some way?  or was it just a dumb idea?
> I personally use it to change the smallseek time to 3 seconds
> since I like to back up a little to hear a word occasionally.
> Please give me some input.
> 
> Danny Rathjens wrote:
> 
> > I found myself editing the values in the source every update so I
> > figured I should make them configurable.
> > diff'd from cvs
> > If this patch makes sense(e.g. option/variable naming) then let me
> > know and I will patch the man page appropriately.
> > 
> > For my next slightly more ambitious project I was thinking of making
> > all of the keyboard definitions configurable(perhaps allow pause and
> > volume control with mouse).
> > My plan is to stumble along blindly until I grow accustomed to the
> > dark.  So I am staying away from the real a/v code until I figure
> > out how it all works and how I could contribute to it, :)
> > 
> 
>       _.,-*~`^'~*-,._ Danny Rathjens _.,-*~`^'~*-,._
> "Do you believe in the Devil?  You know, a supreme evil being dedicated
> to the temptation, corruption, and destruction of man?"
> "I'm not sure man needs the help." --Bill Watterson as Calvin & Hobbes
>