Patch forthcoming that addresses the other concerns.

================
Comment at: clang-apply-replacements/tool/ClangApplyReplacementsMain.cpp:255
@@ +254,3 @@
+    if (!applyReplacements(I->getValue(), NewFileData, Diagnostics)) {
+      errs() << "Failed to apply replacements to " << I->getKey() << "\n";
+      continue;
----------------
Daniel Jasper wrote:
> This should be done via the 'Diagnostics' (and also inside applyReplacements).
There are many places where `errs()` is used instead of Diagnostics currently. 
There's a file-wide FIXME to address this at some point. Beyond the scope of 
this patch at least.


http://llvm-reviews.chandlerc.com/D1730
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to