[PATCH] D72282: [clang-tidy] Add `bugprone-unintended-adl`

Eugene Zelenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 6 11:04:01 PST 2020


Eugene.Zelenko added inline comments.


================
Comment at: clang-tools-extra/docs/clang-tidy/checks/bugprone-unintended-adl.rst:39
+
+   Semicolon-separated list of names that the check ignores. Default is `swap`.
----------------
logan-5 wrote:
> Eugene.Zelenko wrote:
> > Indentation. Please use double back-ticks for swap (or std::swap?).
> Since the 'swap' here is referring to the default value of the option string, shouldn't it just be single backticks?
It was about swap above, not about option default value.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D72282/new/

https://reviews.llvm.org/D72282





More information about the cfe-commits mailing list