jtbandes added inline comments.
Comment at: lib/Format/WhitespaceManager.cpp:650
+ for (unsigned i = 0; i < Newlines; ++i)
+ Text.append(UseCRLF ? " \\\r\n" : " \\\n");
> Note that when you have an empty line, this would turn into:
> #define A \
> int i; \
> \ <-- Note the 1-space indent here.
> int j; \
> int k;
> With my alternative below, that "\" will just be put at column 0, which
> probably isn't better or worse.
I suppose that can be changed back to 1 by using `std::max<int>(1,
EscapedNewlineColumn - 1);` instead, right? I don't have strong feelings about
whether it should be 0 or 1.
cfe-commits mailing list