[PATCH] D42730: [clang-tidy]] Add check for use of types/classes/functions from <functional> header which are deprecated and removed in C++17

[Alexander Kornienko via Phabricator via cfe-commits]

alexfh added inline comments.


================
Comment at: clang-tidy/modernize/ModernizeTidyModule.cpp:50
+    CheckFactories.registerCheck<AvoidFunctionalCheck>(
+        "modernize-avoid-functional");
     CheckFactories.registerCheck<DeprecatedHeadersCheck>(
----------------
aaron.ballman wrote:
> I'm not keen on this name -- it suggests the user should avoid functional, which is certainly not the advice we want to give them. Also, it makes no mention of deprecations, which is what the check is all about.
> 
> How about: `modernize-functional-deprecations` and we can use `modernize-deprecations` as the eventual catch-all umbrella, `modernize-<header name>-deprecations` for other headers, and `modernize-<header name>-deprecations-<api>` if we want to add API-level granularity?
I'd suggest to put "deprecations" first: modernize-deprecations-functional. Or maybe modernize-deprecated-functional?


https://reviews.llvm.org/D42730