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

don hinton via cfe-commits cfe-commits at lists.llvm.org
Thu May 12 11:31:52 PDT 2016


hintonda 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
----------------
Eugene.Zelenko wrote:
> 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.
Ah, got it.  Sorry for being so dense...  Will checkin shortly...


http://reviews.llvm.org/D18575





More information about the cfe-commits mailing list