[MPlayer-dev-eng] [PATCH] -force-key-frames option

[Reimar Döffinger]

On Sun, Oct 17, 2010 at 11:01:45PM +0200, Nicolas George wrote:
> Le sextidi 26 vendémiaire, an CCXIX, Reimar Döffinger a écrit :
> > Could you be more specific? I'm a bit unsure what exactly you mean.
> 
> $ mplayer -dvd-device dvd/1 dvd://1 -chapter invalid
> Invalid chapter range specification invalid
> MPlayer SVN-r32128-4.4.5 (C) 2000-2010 MPlayer Team
> 
> Playing dvd://1.
> <snip>
> 
> I was expecting the invalid chapter range to cause mplayer to exit with an
> error.

As so often, the whole design is undocumented crap.
The problem is that the indicated function is called from the
moption "set" functions - those have no return value and simply
aren't allowed to fail on principle.
There is no way to add a hook into the parse function, which is
the one that is supposed to actually validate the options.
If you feel the motivation to do so you could probably add a new
option type for time value list, but it seems overkill...