thakis added inline comments.
================
Comment at: lib/Rewrite/Rewriter.cpp:455
I->second.write(File.getStream());
+ prewrite(I->first);
}
----------------
amccarth wrote:
> 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.
obsolete now, but while it's true that it's called after the write to the temp
file, it's called before the temp file is moved to the final location -- and
for the client code of this function, the temp file is an implementation detail
it doesn't know about, so it's really called before "the" write.
https://reviews.llvm.org/D30385
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
