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

Stefano Sabatini stefano.sabatini-lala
Wed Sep 12 11:13:24 CEST 2007


On date Thursday 2007-09-06 18:58:49 +0200, Stefano Sabatini encoded:
> On date Thursday 2007-09-06 18:37:51 +0200, Diego Biurrun encoded:
[...] 
> Fixed, new patch attached.

Another auto-review which is in truth just an excuse for a ping, I
also take the occasion to make other two small changes.

[...]
> Index: libavformat/avformat.h
> ===================================================================
> --- libavformat/avformat.h	(revision 10403)
> +++ libavformat/avformat.h	(working copy)
> @@ -794,19 +794,27 @@
>  attribute_deprecated int parse_frame_rate(int *frame_rate, int *frame_rate_base, const char *arg);
>  
>  /**
> - * Converts date string to number of seconds since Jan 1st, 1970.
> - *
> + * Parses \p datestr and returns a corresponding number of microseconds.
> + * @param datestr String representing a date or a duration.
> + * - If not a duration the syntax is:
        ^^^^^^^^^^^^^^^^^
Changed in "If a date".

[...]
>   *  [{YYYY-MM-DD|YYYYMMDD}]{T| }{HH[:MM[:SS[.m...]]][Z]|HH[MM[SS[.m...]]][Z]}
> - * Time is localtime unless Z is suffixed to the end. In this case GMT
> - * Return the date in micro seconds since 1970
> - *
> - * - If a duration:
> + * @endcode
> + * Time is localtime unless Z is appended, in which case it is
> + * interpreted as GMT.
                     ^^^
Changed to UTC.

[...]

Regards.
-- 
Stefano Sabatini
Linux user number 337176 (see http://counter.li.org)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: enhace-parse-date-doc-03.patch
Type: text/x-diff
Size: 1538 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070912/88fd58a6/attachment.patch>



More information about the ffmpeg-devel mailing list