[PATCH] D26511: [clang-tidy] Rename modernize-use-default to modernize-use-equals-default

Malcolm Parsons via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 10 09:27:23 PST 2016

malcolm.parsons added inline comments.

Comment at: clang-tidy/modernize/ModernizeTidyModule.cpp:58
-    CheckFactories.registerCheck<UseDefaultCheck>("modernize-use-default");
+    CheckFactories.registerCheck<UseEqualsDefaultCheck>("modernize-use-equals-default");
aaron.ballman wrote:
> What do we want to do, if anything, for people who have scripts using the old name? Do we want to keep the old name as an alias to the new name for some period of time?
An alias helps if the check was enabled by name, but not if it was disabled by name.
If the alias is temporary, would you want a deprecation warning?
I wouldn't want to warn about `-checks=modernize*`, but maybe warning for `-checks=modernize-use-default` would be useful.


More information about the cfe-commits mailing list