[Ffmpeg-devel] [PATCH] Move libavformat Doxygen and other comments to avformat.h

Panagiotis Issaris takis
Sat Mar 3 21:25:32 CET 2007


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?

With friendly regards,
Takis




More information about the ffmpeg-devel mailing list