[FFmpeg-devel] [RFC] info about stream details

Andreas Öman andreas
Wed Mar 5 14:48:11 CET 2008


Reimar D?ffinger wrote:
> On Wed, Mar 05, 2008 at 01:15:03PM +0100, Andreas ?man wrote:
>> Michael Niedermayer wrote:
>>  > A callback in AVCodec seems to make more sense and is more flexible.
>>
>> Indeed, like this?
>>
> 
>> Index: libavcodec/avcodec.h
>> ===================================================================
>> --- libavcodec/avcodec.h	(revision 12326)
>> +++ libavcodec/avcodec.h	(working copy)
>> @@ -2219,6 +2219,7 @@
>>      void (*flush)(AVCodecContext *);
>>      const AVRational *supported_framerates; ///array of supported framerates, or NULL if any, array is terminated by {0,0}
>>      const enum PixelFormat *pix_fmts;       ///array of supported pixel formats, or NULL if unknown, array is terminanted by -1
>> +    int (*getinfo)(AVCodecContext *, char *buf, size_t buf_size); ///< Write plaintext information about current bitstrem in buf
> 
> Hmm. Naming it get... and the comment saying "Write ..." seems weird to me.

Yeah, it didn't really felt right, and there is also a spelling error.

I'll come up with something for the real patch.





More information about the ffmpeg-devel mailing list