[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