[FFmpeg-devel] [PATCH] Move av_open_input_file probe loop to its own method

Stefano Sabatini stefano.sabatini-lala
Sat Mar 6 21:21:44 CET 2010


On date Saturday 2010-03-06 14:58:38 -0500, Micah F. Galizia encoded:
> On 10-03-05 06:44 AM, Michael Niedermayer wrote:
> >On Thu, Mar 04, 2010 at 10:23:57PM -0500, Micah F. Galizia wrote:
[...]
> Index: libavformat/internal.h
> ===================================================================
> --- libavformat/internal.h	(revision 22246)
> +++ libavformat/internal.h	(working copy)
> @@ -35,4 +35,19 @@
>  void ff_interleave_add_packet(AVFormatContext *s, AVPacket *pkt,
>                                int (*compare)(AVFormatContext *, AVPacket *, AVPacket *));
>  
> +/**
> + * Probe a Bytestream to determine the input format.

Nits: "Probes a bytestream..."

> + *
> + * @param pb The bytestream to probe. It may be closed and opened again.

> + * @param fmt The input format is put here.
> + * @param filename The filename of the stream.
> + * @param logctx The log context
> + * @param offset The offset within the bytestream to probe from.
> + * @param max_probe_size The maximum probe buffer size.

These should not be Upcased (as they are non-complete sentences), also
skip the "." at the end.

> + * @return 0 if OK, AVERROR_xxx otherwise
> + */
> +int ff_probe_input_buffer(ByteIOContext **pb, AVInputFormat **fmt,
> +                          const char *filename, void *logctx,
> +                          unsigned int offset, unsigned int max_probe_size);
> +
>  #endif /* AVFORMAT_INTERNAL_H */

Regards.
-- 
FFmpeg = Fiendish and Faithful Majestic Portentous Elastic Glue



More information about the ffmpeg-devel mailing list