Mesa Can Finally Build With Almost No Compiler Warnings

[phoronix]

Phoronix: Mesa Can Finally Build With Almost No Compiler Warnings

Quite a feat for modern open-source projects with large C/C++ code-bases developed over the years, Mesa3D can almost be compiled now without any warnings -- there's just one remaining...

http://www.phoronix.com/scan.php?page=news_item&px=Mesa-Zeroing-Compiler-Warnings

[pete910]

Eric Engestrom of the open-source Intel team sent out a set of ssix patches

Have we playing with a snake today

[cb88]

Well some of the errors were just silenced but are still there, potentially this is perfectly fine though.

[tildearrow]

Have we playing with a snake today

The snake has eaten me asleep.

[dungeon]

Might try it, with Sid's stable weekly rolling compiler... hopefully there would be no warnings on any architecture

Snake's dream

[Guest]

They should setup CI, build with Wall, Werror, pedantic, flags. And, if PR's build doesn't succeed, then PR doesn't get in... That would keep project clean, as it's easy to fix things immediately, but TODOs usually get almost never resolved...

[varikonniemi]

Yes, i am amazed at the nonchalant attitude towards warnings. A stable release should not happen with one. Either change the code, or the compiler.

[schmidtbag]

Yes, i am amazed at the nonchalant attitude towards warnings. A stable release should not happen with one. Either change the code, or the compiler.

I agree. Warnings exist for a reason, so stability (or just simply the quality of code) is very questionable if they are left un-attended.

[zman0900]

Yes, i am amazed at the nonchalant attitude towards warnings. A stable release should not happen with one. Either change the code, or the compiler.

I bet a big part of that is code that originally produced no warnings, but newer compilers are better at finding problems / have more warnings.

[varikonniemi]

Then fix it before your next release. Either in your code, or in the official compiler you use. GCC gives warnings when it should not? Use clang.