Kostya Shishkov <[email protected]> writes: > 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?
I care about documentation not placing false restrictions on usage. -- Måns Rullgård [email protected] _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
