> if (OPT_Wmissing_braces) > warning (OPT_Wmissing_braces, "missing braces around > initializer");
FYI OPT_Wmissing_braces is an enum constant; it will always be nonzero. > [3] > warning (OPT_Wmissing_braces, "missing braces around initializer"); That is what we decided to do. Note, however, if the logic required to determine if a warning is warranted is sufficiently complex, *also* checking the variable is considered an optimization: if (warn_missing_braces && expensive_warning_check()) warning (OPT_Wmissing_braces, "..."); In this case, the warn_missing_braces check is done only to avoid calling expensive_warning_check() if we know in advance we won't be needing it.