[PATCH] Refactor: Move remove-cstr-calls from a standalone executable to a clang-tidy check

Alexander Kornienko alexfh at google.com
Mon Feb 23 06:02:12 PST 2015


In http://reviews.llvm.org/D7318#126692, @LegalizeAdulthood wrote:

> In http://reviews.llvm.org/D7318#125262, @mclow.lists wrote:
>
> > I don't have a problem with making this a clang-tidy module, but it was (also) meant as an example of how to write a refactoring tool.
>
>
> Do we now have enough examples of refactoring tools that the example motivation is no longer a concern?


I didn't find any references to remove-cstr-calls from docs, so I don't think the tool is in any sense special. There is a `clang-rename` which can be used as an example of a refactoring tool, and there is a dedicated `tool-template`. IMO, these should be enough.


http://reviews.llvm.org/D7318

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list