[FFmpeg-devel] [PATCH] Remove unused code
Måns Rullgård
mans
Sat May 2 00:54:11 CEST 2009
Michael Niedermayer <michaelni at gmx.at> writes:
> On Fri, May 01, 2009 at 11:50:55PM +0200, Benjamin Larsson wrote:
>> Dead assignments found by CSA.
>>
>> Index: libavcodec/ratecontrol.c
>> ===================================================================
>> --- libavcodec/ratecontrol.c (revision 18726)
>> +++ libavcodec/ratecontrol.c (working copy)
>> @@ -754,7 +754,6 @@
>> rcc->mv_bits_sum[pict_type] += rce->mv_bits;
>> rcc->frame_count[pict_type] ++;
>>
>> - bits= rce->i_tex_bits + rce->p_tex_bits;
>> rate_factor= rcc->pass1_wanted_bits/rcc->pass1_rc_eq_output_sum * br_compensation;
>>
>> q= get_qscale(s, rce, rate_factor, picture_number);
>
> i think, i didnt apply this because this makes bits more correct even
> if unused.
> not a big issue though ...
If it is not used, how can you say anything about its correctness?
--
M?ns Rullg?rd
mans at mansr.com
More information about the ffmpeg-devel
mailing list