[PATCH] D28315: Update tools to use new getStyle API

Antonio Maiorano via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 11 05:31:13 PST 2017


amaiorano added inline comments.


================
Comment at: change-namespace/ChangeNamespace.cpp:892
+      llvm::errs() << llvm::toString(Style.takeError()) << "\n";
+      continue;
+    }
----------------
amaiorano wrote:
> ioeric wrote:
> > I'd still like to apply replacements that are not cleaned up. Could you add the following line before continue, thanks! :)
> > ```
> > FileToReplacements[FilePath] = Replaces;
> > ```
> Will do.
Ah darn, just realized that I forgot to make this change you asked for @ioeric. Let me know if everything else is okay, and I'll be sure to add this in before submitting. 


https://reviews.llvm.org/D28315





More information about the cfe-commits mailing list