klimek added inline comments.
> MigrateTool.cpp:52 > + // Get all files that are affected by the migration, i.e. users of the > symbol. > + auto Files = > Env.getAffectedFilesFinder().getAffectedFiles(Spec.getOldName()); > + if (!Files) 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(...) https://reviews.llvm.org/D24380 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits