[PATCH] D28235: [clang-format cleanup] merge continuous deleted lines into one deletion.
Paweł Żukowski via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sat Feb 4 08:20:11 PST 2017
idlecode added inline comments.
================
Comment at: lib/Format/Format.cpp:1182
+ I != E; ++I)
+ LastToNextLineFirst[(*I)->Last] = (*std::next(I))->First;
tooling::Replacements Fixes;
----------------
Could you add brackets around loop body?
================
Comment at: lib/Format/Format.cpp:1184
tooling::Replacements Fixes;
std::vector<FormatToken *> Tokens;
std::copy(DeletedTokens.begin(), DeletedTokens.end(),
----------------
While you are here, could you get rid of `Tokens` vector too?
I believe iterating over `DeletedTokens`directly will do the job.
https://reviews.llvm.org/D28235
More information about the cfe-commits
mailing list