[Ffmpeg-devel] [PATCH] Move libavformat Doxygen and other comments to avformat.h
Panagiotis Issaris
takis
Sat Mar 3 22:49:03 CET 2007
Michael Niedermayer schreef:
> Hi
>
> On Sat, Mar 03, 2007 at 09:25:32PM +0100, Panagiotis Issaris wrote:
>
>> Panagiotis Issaris schreef:
>>
>>> Hi,
>>>
>>> Michael Niedermayer schreef:
>>>
>>>> On Thu, Mar 01, 2007 at 05:12:46PM +0100, Panagiotis Issaris wrote:
>>>>
>>>>
>>>>> +/**
>>>>> + * Print nice hexa dump of a buffer
>>>>> + * @param f stream for output
>>>>> + * @param buf buffer
>>>>> + * @param size buffer size
>>>>> + */
>>>>> void av_hex_dump(FILE *f, uint8_t *buf, int size);
>>>>> +
>>>>> +/**
>>>>> + * Print on 'f' a nice dump of a packet
>>>>> + * @param f stream for output
>>>>> + * @param pkt packet to dump
>>>>> + * @param dump_payload true if the payload must be displayed too
>>>>> + */
>>>>> void av_pkt_dump(FILE *f, AVPacket *pkt, int dump_payload);
>>>>>
>>>>>
>>>> these should be changed to av_logish style, (FILE dependancy sucks ...)
>>>> thats of course seperate and independant of this patch
>>>>
>>>>
>>> OK, I'll write a patch for this later today.
>>>
>> Do you prefer to be able to specify the level as a av_(hex|pkt)_dump
>> parameter, or should AV_LOG_DEBUG be implied?
>>
>
> id like to be able to specify the level
>
Okay, I'll change that as soon as my other question regarding the patch
is answered.
With friendly regards,
Takis
More information about the ffmpeg-devel
mailing list