2 ms·
They are not 'visible' if you just let them scroll by. If there are individual warnings that have no meaning, turn them off. Everything else, is help from the
by outsomnia 5y ago
They are not 'visible' if you just let them scroll by.
If there are individual warnings that have no meaning, turn them off.
Everything else, is help from the compiler, which is your only friend. Ignoring even one of those should not be tolerated at all. They must be analyzed and the code improved to remove them. And you enforce your team following that policy with -Werror.
- daemin 5y agoYeah, but push comes to shove during shipping, people leave, and the pragmas remain in the code in various places. At least if they are scrolling by then people can get fed up with that and take some initiative to fix those warnings. Or the new people joining your studio can come in and try to fix them. It's much more difficult to get people to fix pragmas scattered throughout the code. Which ones were legit, which ones were put in at the last moment out of frustration, which ones were put in temporarily years ago? I think a softer approach works better since people won't be so quick to work around the compiler by disabling the warnings.