Re: [PATCH] D21601: Make tooling::applyAllReplacements return llvm::Expected instead of empty string to indicate potential error.

[Manuel Klimek via cfe-commits]

klimek added a comment.

In http://reviews.llvm.org/D21601#466513, @ioeric wrote:

> Would EXPECTED_FALSE(!Code) be better?


Not really. Can we at least use static_cast<bool>(...)?


http://reviews.llvm.org/D21601



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits