[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