On Sat, Feb 13, 2021 at 12:48 AM sfan5 <sf...@live.de> wrote:
>
> 12.02.21 - 21:43 - Andreas Rheinhardt:
> > sfan5:
> >> Hi,
> >>
> >>
> >> attached v2 patch after discussion on IRC with JEEB (as he already
> >> mentioned).
> >>
> >> Only change is that the log level turns to debug when missing parameter
> >> sets are within spec (cf. 14496-15).
> >>
> >>
> >> - av_log(avctx, AV_LOG_ERROR, "Could not extract PPS/SPS from
> >> extradata");
> >> - ret = AVERROR_INVALIDDATA;
> >> + const int warn = is_avc && avctx->codec_tag !=
> >> MKTAG('a','v','c','1') &&
> >> + avctx->codec_tag != MKTAG('a','v','c','2');
> >> + av_log(avctx, warn ? AV_LOG_WARNING : AV_LOG_DEBUG,
> >> + "Could not extract PPS/SPS from extradata\n");
> >> + ret = 0;
> >> }
> > warn = is_avc && (avctx->codec_tag == MKTAG('a','v','c','1') ||
> > avctx->codec_tag == MKTAG('a','v','c','2')
> > is what you (should) want.
> >
> > - Andreas
>
> Thanks for pointing that out, you're correct.
>
> here's v3:
After some brief discussion on IRC, this looks good to me :) .
Jan
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
