On 06/16/2018 11:00 AM, Ulrich Wilkens wrote:

The latest patches to hide warnings under BSDs (55b4e2 and f84035) are
raising a lot of new problems.
They don't support compiling with gcc. Either the build is totally broken
or new warnings are raised.

I tested with clang - the default compilers... Perhaps older ones might have a problem though...

Even with older clang there are many new warnings.
The reason is that the new compiler options are just not supported.

I could send a patch but it would use some of the __GNUC* and __clang*
macros. It makes it complicated and is not very nice.

By the way: I don't like this sort of "solution". It indeed reduces some
noise, but also possibly hides important information.



I agree WRT to potentially hiding a newly introduced warning, but how would you even spot that among 9000+ others? It's just noise.

It's not intended as a "solution", rather as a filter.

If you would like to revert the problematic options, feel free, but leave the working ones there. And please *do not* re-add -Wall. That's just pointless for an end user.

If someone wants to go through and clean some of these thousands of issues up, they will know how to supply the proper -W options.

--
Jon Trulson

"Fire all weapons and open a hailing frequency for my victory yodle."

                              - Zapp Brannigan

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
cdesktopenv-devel mailing list
cdesktopenv-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/cdesktopenv-devel

Reply via email to