[FFmpeg-devel] [PATCH 29/61] avcodec/mobiclip: Avoid unnecessary VLC structure
Andreas Rheinhardt
andreas.rheinhardt at outlook.com
Thu Sep 28 02:44:13 EEST 2023
Michael Niedermayer:
> On Wed, Sep 27, 2023 at 12:17:00AM +0200, Andreas Rheinhardt wrote:
>> Everything besides VLC.table is basically write-only
>> and only VLC.table needs to be retained.
>>
>> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
>> ---
>> libavcodec/mobiclip.c | 41 ++++++++++++++++++-----------------------
>> 1 file changed, 18 insertions(+), 23 deletions(-)
>
> This produces (vissible) difference in some files
>
> for example https://samples.ffmpeg.org/game-formats/moflex/B7.moflex
> (this doesnt seem to decode correctly before either but it looks
> worse here after this change)
>
Correct, I am initializing rl_vlc[1} with the wrong symbols table.
Thanks for spotting this.
- Andreas
More information about the ffmpeg-devel
mailing list