On Fri, Sep 23, 2011 at 01:40:56PM +0100, Måns Rullgård wrote:
> 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.

Like autoselecting parallel frame decoding but who cares?
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to