yvvan marked an inline comment as done.
yvvan added inline comments.
================
Comment at: clang/include/clang/Tooling/ReplacementsYaml.h:35
+ : FilePath(""), Offset(0), Length(0), ReplacementText("") {
+ size_t lineBreakPos = ReplacementText.find('\n');
+ while (lineBreakPos != std::string::npos) {
----------------
gribozavr wrote:
> Sorry, I don't understand how this works -- ReplacementText does not contain
> a \n, so lineBreakPos will be npos, and the loop below will not execute...
Quite opposite. This patch targets the cases where it's not npos (see the test
example). So it has a line break and this line break should be transformed into
two line breaks.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D63482/new/
https://reviews.llvm.org/D63482
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
