On Tue, Feb 04, 2014 at 09:54:41PM +0100, Anton Khirnov wrote: > --- a/libavformat/avformat.h > +++ b/libavformat/avformat.h > @@ -173,6 +173,58 @@ > + * > + * - the @ref AVFormatContext.oformat "oformat" field must be set to select > the > + * muxer that will be used; > + * - unless the format is of the AVFMT_NOFILE type, the @ref > AVFormatContext.pb > + * "pb" field must be set to an opened IO context, either returned from > + * avio_open2() or a custom one; > + * - unless the format is for the AVFMT_NOSTREAMS type, at least one stream > must > + * be created with the avformat_new_stream() function. The caller should > fill > + * the @ref AVStream.codec "stream codec context" information, such as the > + * codec @ref AVCodecContext.codec_type "type", @ref > AVCodecContext.codec_id > + * "id" and other parameters (e.g. width / height, the pixel or sample > format, > + * etc.) as known. The @ref AVCodecContext.time_base "codec timebase" > should > + * be set to the timebase that the caller desires to use for this stream > (note > + * that the timebase actually used by the muxer can be different, as will > be > + * described later). > + * - the caller may fill in additional information, such as @ref > + * AVFormatContext.metadata "global" or @ref AVStream.metadata "per-stream" > + * metadata, @ref AVFormatContext.chapters "chapters", @ref > + * AVFormatContext.programs "programs", etc. as described in the > + * AVFormatContext documentation. Whether such information will actually be > + * stored in the output depends on what the container format and the muxer > + * support.
Capitalize the first sentences of each point, otherwise LGTM. Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
