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

Micah F. Galizia micahgalizia
Sat Mar 6 21:34:27 CET 2010


On 10-03-06 03:21 PM, Stefano Sabatini wrote:
> 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.

CAPS and grammar fixed.
-- 
Micah F. Galizia
micahgalizia at gmail.com

"The mark of an immature man is that he wants to die nobly for a cause, 
while the mark of the mature man is that he wants to live humbly for 
one."   --W. Stekel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: probe_buffer3.diff
Type: text/x-diff
Size: 5372 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100306/b1fe77de/attachment.diff>



More information about the ffmpeg-devel mailing list