[FFmpeg-devel] [PATCH 23/29] avcodec/mpeg12dec: use ff_frame_new_side_data

Anton Khirnov anton at khirnov.net
Thu Mar 7 14:37:17 EET 2024


Quoting James Almer (2024-03-07 13:25:07)
> I wouldn't. It adds an extra layer of abstraction for no real gain.

The gain is simpler API. What's the gain in forcing decoder authors to
call a different function to accomplish the same task, just because
frame threading happens to be used.

> And the name in this case is very specific and self explanatory in how
> it's different from the alternative.

>From decoder POV it's not different at all, it does exactly the same
thing.

-- 
Anton Khirnov


More information about the ffmpeg-devel mailing list