[PATCH] D17761: Added applyAllReplacementsAndFormat that works for multiple files.

Daniel Jasper via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 1 21:47:57 PST 2016


djasper added a comment.

I don't think this is the right abstraction:

- Formatting so far is fundamentally per file. Grouping replacements per file is something that can be done outside of libFormat.
- The logic to sort and deduplicate Replacements is important independent of whether formatting is done on them. So this should also go outside of libFormat.


http://reviews.llvm.org/D17761





More information about the cfe-commits mailing list