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

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 28 05:36:49 PST 2020


aaron.ballman added inline comments.


================
Comment at: clang-tools-extra/clang-tidy/bugprone/UnintendedADLCheck.h:27
+  const bool IgnoreOverloadedOperators;
+  const std::vector<std::string> Whitelist;
+
----------------
I'd prefer a more descriptive name for this (and the user-facing option + documentation). How about `AllowedIdentifiers` or `ADLEnabledIdentifiers` or some such?


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