[PATCH] D18575: [clang-tidy] New checker to replace deprecated throw() specifications

Eugene Zelenko via cfe-commits cfe-commits at lists.llvm.org
Thu May 12 11:26:22 PDT 2016


Eugene.Zelenko added inline comments.

================
Comment at: docs/clang-tidy/checks/modernize-use-noexcept.rst:32
@@ +31,3 @@
+``noexcept(false)``.  Additinally, users can also use
+:option:``ReplacementString`` to specify a macro to use instead of
+``noexcept``.  This is useful when maintaining source code that must
----------------
hintonda wrote:
> Eugene.Zelenko wrote:
> > Sorry, missed last time. Please use ` for check option and its value(s). Same for last line in file.
> I'm not sure I understand what you want me to change.  Did you want me to append that phrase, or replace something?
Replace :option:``ReplacementString`` with :option:`ReplacementString`. Same in last line with option and NOEXCEPT.

'' is for language constructs, ' is for command line options and their parameters.


http://reviews.llvm.org/D18575





More information about the cfe-commits mailing list