[FFmpeg-devel] [PATCH 1/3] avutil/frame: add av_frame_replace

[James Almer]

On 8/13/2021 12:22 PM, Andreas Rheinhardt wrote:
> James Almer:
>> On 8/10/2021 6:59 PM, James Almer wrote:
>>> Signed-off-by: James Almer <jamrial at gmail.com>
>>> ---
>>> Not going to bother with implementing replace for side data, since
>>> it's IMO not
>>> worth it, but patches are welcome of course.
>>>
>>> Missing version bump and APIchanges entry.
>>>
>>>    libavutil/frame.c | 98 +++++++++++++++++++++++++++++++++++++++++++++++
>>>    libavutil/frame.h | 18 +++++++++
>>>    2 files changed, 116 insertions(+)
>>
>> Will apply the set if there are no objections.
> 
> How many allocations are saved by this normally? (I.e. with a random
> H.264 file you have)?

I looked at a VP9 file with patch 4/4, and it seemed to remove about a 
third of the overall allocations.

> 
> - Andreas
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
>