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

[Nicolas George]

Le sextidi 26 vendémiaire, an CCXIX, Reimar Döffinger a écrit :
> 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...

It would probably be more useful to ensure that the return code from the
CONF_TYPE_FUNC_PARAM handlers is properly taken into account. But that is
for sometime else.

I just committed the first part.

Are there any other remarks on the second? And anything on parts 4 and 5?

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20101018/018a2586/attachment.pgp>