srhines marked an inline comment as done. srhines added inline comments.
================ Comment at: lib/Tooling/Core/Replacement.cpp:505 - assert(!Err && - "Replacements must not conflict since ranges have been merged."); - llvm::consumeError(std::move(Err)); ---------------- hintonda wrote: > While obviously correct, are you concerned that by removing the explanatory > text, this change will obscure the reason for the assert? The text is now in a comment above the call. https://reviews.llvm.org/D36806 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
