On 04/05/2018 07:28 PM, Martin Sebor wrote:
Attached is the final version of the patch to adjust the lists of options (C++ Language Options and -Wall) to include missing C++ options, reference the forms of options that aren't the default, and use TexInfo tables for the lists of options in -Wall and -Wextra to address Nathan's comment. The patch also fixes bug 71283.
-Wnoexcept -Wnoexcept-type -Wclass-memaccess @gol
...
+-Wclass-memaccess -Wclobbered -Wcomment -Wconditionally-supported @gol
-Wclass-memaccess is already in the C++ options summary, I don't think we need to also add it to the diagnostic options summary.
-@item -Wclass-memaccess @r{(C++ and Objective-C++ only)} +@item -Wclass-memaccess @r{(C++ only)}
You removed "and Objective-C++" only for this option? Let's either change all of them or none.
-@item -Wsubobject-linkage @r{(C++ and Objective-C++ only)} +@item -Wno-subobject-linkage @r{(C++ and Objective-C++ only)} @opindex Wsubobject-linkage @opindex Wno-subobject-linkage Warn if a class type has a base or a field whose type uses the anonymous
-@item -Wdelete-incomplete @r{(C++ and Objective-C++ only)} +@item -Wno-delete-incomplete @r{(C++ and Objective-C++ only)} @opindex Wdelete-incomplete @opindex Wno-delete-incomplete Warn when deleting a pointer to incomplete type, which may cause
If you're reversing the sense of the flag, please adjust the documentation to match.
Jason