[FFmpeg-devel] [PATCH] Optimization of original IFF codec

[Måns Rullgård]

Sebastian Vater <cdgs.basty at googlemail.com> writes:

> Hey Mans!
>
> M?ns Rullg?rd a ?crit :
>> No, that's not what it means.  A local variable with a const qualifier
>> means simply that the value of the variable cannot be changed after
>> the initialisation.  If an attempt is made to assign a value to it
>> later, the compiler must issue an error.  Other than that it has no
>> effect at all.
>>   
>> Perhaps you confused it with "static const" symbols, which indeed have
>> a fixed value although their scope is limited to the enclosing block.
>> These, however, must be initialised by a constant expression.
>>   
> Well, I wanted I declare it static const first, but then I realized that
> it will never be used outside the function, so I dropped the static
> before it and kept it just const, but, of course, I know what const is.
>
> I wanted it that way that b can't be changed after initialization, it
> would even break the loop if someone would change it in the loop afterwards.
>
> The question though, was that I should remove the const modifier or not
> and we're waiting for you to decide this.

That's a matter of taste.  I don't care.

-- 
M?ns Rullg?rd
mans at mansr.com