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

Nicolas George nicolas.george at normalesup.org
Sun Oct 17 14:35:11 CEST 2010


Le sextidi 26 vendémiaire, an CCXIX, Reimar Döffinger a écrit :
> If even the linux man page tells you not to rely on it I'd say
> it must be very common.
> Note that not even Windows (using msvcrt) claims to comply to either
> of the standards you quote.

All right. Attached, a new version that uses ">= 3" instead of "== 3", which
should work in both cases.

> For the current use cause even that is overkill.
> For that you just need to check that the following char is either 0 or ','.
> You do not need to return a length or change the function in any other way,
> and just need to add a strchr - in the current usage (if you want to fix them)
> to check that there is no ',', in the new usages to advance to the next time.

I am not sure what your point really is.

The version in my current proposal:

- returns the length, or 0 for error, instead of the ugly magical -1E100;

- if the third argument is 0, it returns an error if there is anything after
  the timestamp;

- if the third argument is ',', it returns an error if there is anything
  after the timestamp except a comma.

Is it ok?

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mplayer-forcekeyframes-20101017-1427-01-parser.diff
Type: text/x-diff
Size: 2254 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20101017/e9abb475/attachment.diff>
-------------- 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/20101017/e9abb475/attachment.pgp>


More information about the MPlayer-dev-eng mailing list