[FFmpeg-devel] [PATCH 12/39] lavc/ffv1: drop redundant FFV1Context.quant_table

[James Almer]

On 7/17/2024 7:37 PM, Michael Niedermayer wrote:
> On Tue, Jul 16, 2024 at 07:11:27PM +0200, Anton Khirnov wrote:
>> In all cases except decoding version 1 it's either not used, or contains
>> a copy of a table from quant_tables, which we can just as well use
>> directly.
>>
>> When decoding version 1, we can just as well decode into
>> quant_tables[0], which would otherwise be unused.
>> ---
>>   libavcodec/ffv1.h    |  1 -
>>   libavcodec/ffv1dec.c | 10 +++++++---
>>   libavcodec/ffv1enc.c |  6 ++----
>>   3 files changed, 9 insertions(+), 8 deletions(-)
> 
> the code is 1 line longer and using the first entry as a special

Two lines are comments...

> case doesnt seem simpler
> 
> thx
> 
> [...]
> 
> 
> _______________________________________________
> 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".