Re: Dewarningization patch



Miguel:
>> I do not have any objections to using -Werror, but last I checked
>> ORBit was still generating broken C code (with the high warning levels
>> they would report warnings that apparently are not harmful).
>> 
>> If this has been taken care of, please go ahead and add -Werror.  I
>> just remember having to fight Nautilus to work due to this ORBit
>> error.

This has been taken care of for Nautilus by adding -Wno-uninitialized to the
warning options for now.

Maciej:
> Well, I successfully compiled Bonobo with -Werror on after applying my
> patch, so I don't think this is an issue.

Beware, Maciej. The warning in question is an uninitialized variable warning
that only shows up on optimized builds. You might get a false sense of
security if you have only done builds with "-g" and no "-O".

    -- Darin



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]