[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
> 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