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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to