[PATCH] D30385: clang-format: Don't leave behind temp files in -i mode on Windows, PR26125, reloaded

Adrian McCarthy via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 27 13:15:07 PST 2017


amccarth added inline comments.


================
Comment at: lib/Rewrite/Rewriter.cpp:455
       I->second.write(File.getStream());
+      prewrite(I->first);
     }
----------------
I'm not familiar with the structure of this code, but it seems odd to me that the callback is called `prewrite` when it happens _after_ the write.


https://reviews.llvm.org/D30385





More information about the cfe-commits mailing list