[PATCH] [Commented On] D24380: [migrate-tool] Framework for a codebase-dependent migration tool.

Eric Liu via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 30 02:21:48 PDT 2016


ioeric added inline comments.


> klimek wrote in MigrateTool.cpp:52
> I'm wondering whether we really want to evolve this to a callback / executor based design. That is, have AffectedFilesFinder give an interface like RunOnAffectedFiles(...)

That sounds like a good design, but I'm not sure if we need this at this point since actions on affected files would simply be renaming and include fixing now and in the foreseeable future.

https://reviews.llvm.org/D24380





More information about the cfe-commits mailing list