[FFmpeg-devel] [PATCH] avcodec/frame_thread_encoder: check for frame threading codec cap instead of intra only

James Almer jamrial at gmail.com
Mon May 18 18:06:30 EEST 2020


On 5/18/2020 11:36 AM, Anton Khirnov wrote:
> Quoting James Almer (2020-05-18 15:56:36)
>> It's the correct dedicated capability reported by supported encoders.
>> Otherwise, the frame thread path will be used for unsupported encoders
>> like r210 for no gain.
>>
>> Signed-off-by: James Almer <jamrial at gmail.com>
>> ---
> 
> Looks ok.
> 
> Also, this capability looks like it should not exist since it's not an
> encoder capability, it's a codec property. Same for CAP_LOSSLESS.

True. There's a AVCodecDescriptor.props flag for both of them after all.

Also, after this change AV_CODEC_CAP_INTRA_ONLY is unused (not counting
cmdutils.c where it's used to print an entry in a log message), so we
could safely deprecate it and remove it.


More information about the ffmpeg-devel mailing list