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

[Manuel Klimek via cfe-commits]

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