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

Reply via email to