diff options
author | Andreas Rheinhardt <andreas.rheinhardt@outlook.com> | 2025-06-13 00:08:20 +0200 |
---|---|---|
committer | Andreas Rheinhardt <andreas.rheinhardt@outlook.com> | 2025-06-21 22:08:52 +0200 |
commit | 58a222f561f36d8b5c83b6758f94c8a65e96cb4c (patch) | |
tree | aa1b25c76ea8611c9935e4285b71ea2cb5acc168 /tests | |
parent | fbb8e605ed8ad72dd134a5202ac17084ccd1addb (diff) | |
download | ffmpeg-58a222f561f36d8b5c83b6758f94c8a65e96cb4c.tar.gz |
avcodec/svq1enc: Stop calling ff_mpv_common_init()
It is not really needed, as none of the buffers allocated
in ff_mpv_common_init() are used lateron; in fact, ff_mpv_common_init()
is called with MpegEncContext.width and height equal to zero
(as well as encoding equal to zero, so that the error resilience
parts are initialized...), so that all the buffers are too small
anyway.
The call to ff_mpv_common_init() has been added in commit
f6774f905fb3cfdc319523ac640be30b14c1bc55 in order to allocate
{current,last,new}_picture.f (an AVFrame). Yet this is unnecessary
since 7814dd77aa61703e3d43fc72bfaf6a9fbc42ff9c.
The only task that ff_mpv_common_init() does that may be used
is initializing the HpelDSPContext embedded in the MpegEncContext.
In fact, it was not initialized before
f6774f905fb3cfdc319523ac640be30b14c1bc55 and not initializing
it passes FATE, yet I can't prove that it is indeed unnecessary.
Therefore it is still initialized and used instead of
SVQ1EncContext.hdsp.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Diffstat (limited to 'tests')
0 files changed, 0 insertions, 0 deletions