[Ffmpeg-devel] Compiler warnings

Måns Rullgård mru
Thu Feb 2 02:45:38 CET 2006


Michael Niedermayer <michaelni at gmx.at> writes:

> Hi
>
> On Wed, Feb 01, 2006 at 09:03:36AM +0000, Dieter wrote:
>> >> It is good practice to keep harmless warnings away.  That way, the
>> >> real ones are more easily noticed.
>> 
>> YES!
>> 
>> > The easy way to silence the rest of the warnings is to supply
>> > the -Wno-pointer-sign option to gcc
>> 
>> Can you guarantee that every possible pointer-sign warning
>> now and in the future will be harmless?  
>
> no, can you gurantee that every odd numbered line will be bug free? so 

Sure, make them blank.

> why not print a warning for them? its not about that it might in some
> very rare cases indicate a bug but if it does more often then the
> remaning lines (i can show you cases where warining free lines where
> buggy, can you show me one where a f([(signed)]p) case was buggy)
> and MOST important are you checking the code to see if its buggy before
> you add (signed)/(unsigned) casts or are you maybe just blindly 
> suppressing the warnings without understanding the code
> now tell me how THAT is better from suppressing the warnings with a
> compiler switch ...

I can think of cases where a signed/unsigned difference could cause
unexpected behavior, but these are rare.

-- 
M?ns Rullg?rd
mru at inprovide.com





More information about the ffmpeg-devel mailing list