Giovanni Bajo wrote:
Actually, the point is that you *never* need to explicitally name the
"warn_<foo>" variable unless you are optimizing. In other words, code which
presently is:
if (warn_foo
&& frobbed)
warning (0, "foo is frobbed");
should be replaced with:
if (frobbed)
warning (OPT_Wfoo, "foo is frobbed");
Perhaps a macro should be defined:
maybe_warning (OPT_Wfoo, frobbed, "foo is frobbed");
This tests whether Wfoo is enabled *before* it evaluates frobbed.
--
--Per Bothner
[EMAIL PROTECTED] http://per.bothner.com/
