On Mon, Apr 24, 2017 at 11:25:43AM -0300, James Almer wrote:
> On 4/24/2017 11:13 AM, Diego Biurrun wrote:
> > On Sat, Apr 22, 2017 at 01:05:01PM +0200, Diego Biurrun wrote:
> >> --- /dev/null
> >> +++ b/libavcodec/clearvideo.c
> >> @@ -0,0 +1,387 @@
> >> +static int clv_decode_frame(AVCodecContext *avctx, void *data,
> >> + int *got_frame, AVPacket *avpkt)
> >> +{
> >> + if (frame_type & 0x2) {
> >> + } else {
> >> + /* Only I-frames are supported for now. */
> >> + }
> >
> > Here I might use avpriv_report_missing_feature() and return
> > AVERROR_PATCHWELCOME instead. This would of course result in a lot
> > of console spam. Thoughts?
>
> Add an did_warn variable to CLVContext, and set it to true after the
> warning is printed the first time.
Good idea, thanks.
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
