[FFmpeg-devel] [REQUEST] avcodec/scpr: revert d70276921348

[James Almer]

On 8/20/2019 2:46 PM, Michael Niedermayer wrote:
> On Tue, Aug 20, 2019 at 02:49:13PM +0200, Paul B Mahol wrote:
>> Hi,
>>
>> I kindly ask that following commit: d702769213487923c0fb0abe4b61f4d9ebddb88b
>> is reverted as it does contradicts with reference decoder.
> 
> Noone is listed in MAINTAINERs for scpr.c
> the main author of scpr.c is you, so  i think you are the closest to a
> maintainer.
> 
> So if after everyone who wants to comment has commented you still want this
> reverted. Then ill revert (or you can revert too)
> but personally i think that skiping duplicated frames has many advantages
> and its better to skip them
> 
> Thanks

I suggested on IRC to add an option somewhere (new AVCodecContext field?
ffmpeg cli option? Mapping it to AV_CODEC_FLAG2_FAST?) to tell crf
decoders to instead output vrf in situations like this.

Paul didn't like the idea, but i still think it's worth considering.