On Tue, Feb 04, 2014 at 09:54:42PM +0100, Anton Khirnov wrote:
> --- a/libavformat/avformat.h
> +++ b/libavformat/avformat.h
> @@ -897,36 +906,53 @@ typedef struct AVFormatContext {
> +    /**
> +     * input or output filename
> +     *
> +     * - demuxing: set by avformat_open_input()
> +     * - muxing: may be set by the caller before avformat_write_header()     
> *
> +     */
> +    char filename[1024];

Stray trailing star.

> @@ -989,8 +1018,8 @@ typedef struct AVFormatContext {
>       * Demuxers for which a full in-memory index is mandatory will ignore
>       * this.
> -     * muxing  : unused
> -     * demuxing: set by user
> +     * - muxing  : unused
> +     * - demuxing: set by user

The aligned colons look slightly silly IMO.

> @@ -1008,41 +1037,49 @@ typedef struct AVFormatContext {
>       * in the trailer.  To write chapters in the trailer, nb_chapters
>       * must be zero when write_header is called and non-zero when
>       * write_trailer is called.
> -     * muxing  : set by user
> -     * demuxing: set by libavformat
> +     * - muxing  : set by user
> +     * - demuxing: set by libavformat

same

>      /**
>       * Start time of the stream in real world time, in microseconds
> -     * since the unix epoch (00:00 1st January 1970). That is, pts=0
> -     * in the stream was captured at this real world time.
> -     * - encoding: Set by user.
> -     * - decoding: Unused.
> +     * since the unix epoch (00:00 1st January 1970). That is, pts=0 in the
> +     * stream was captured at this real world time.
> +     * Muxing only, set by the caller before avformat_write_header().
>       */

Unix

LGTM otherwise

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to