Diego Biurrun <[email protected]> writes:

> On Fri, Sep 23, 2011 at 02:00:21PM +0200, [email protected] wrote:
>> From: Benjamin Larsson <[email protected]>
>
> I suggest the following log message:
>
>   Add codec capability to mark a codec as intra-only.
>
>   This eliminates the need for the is_intra_only() helper function.
>
>> --- a/libavcodec/avcodec.h
>> +++ b/libavcodec/avcodec.h
>> @@ -710,6 +710,10 @@ typedef struct RcOverride{
>> +/**
>> + * Codec is intra only, used to mark packets when some containers miss that 
>> info
>> + */
>> +#define CODEC_CAP_INTRA_ONLY       0x4000
>
> /**
>  * Codec is intra-only. Used to mark packets for containers which lack
>  * that information.
>
> LGTM otherwise, nice idea.

Telling people what to use it for is wrong.  What you describe above is
*one* possible use.  I can easily imagine others.

-- 
Måns Rullgård
[email protected]
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to