[FFmpeg-devel] [PATCH] Enhace documentation for libavformat/utils.c:parse_date

[Michael Niedermayer]

Hi

On Wed, Sep 12, 2007 at 11:13:24AM +0200, Stefano Sabatini wrote:
[...]
> + * Returns the number of microseconds since 1st of January, 1970 up
> + * to the time of the parsed date or up to the current time if \p
> + * datestr cannot be successfully parsed.

i do not think that this is the intended behavior but rather a "bug"
in the implementation thus you change the API by hardcoding the bug in
it and would need to bump the respective versions but i wont accept that

using the current date in case a string cannot be parsed is a very odd
way to handle an error

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Breaking DRM is a little like attempting to break through a door even
though the window is wide open and the only thing in the house is a bunch
of things you dont want and which you would get tomorrow for free anyway
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070912/32c907da/attachment.pgp>