[PATCH] D125961: [clang-format] Don't break lines after pragma region
Marek Kurdej via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri May 20 04:59:01 PDT 2022
curdeius requested changes to this revision.
curdeius added a comment.
This revision now requires changes to proceed.
Thanks for the patch!
Could you please add a unit test in unittest/Format/FormatTest.cpp instead of in lit-based test/?
Is there a bug report that your patch fixes?
================
Comment at: clang/lib/Format/TokenAnnotator.cpp:1228
bool IsMark = CurrentToken->is(Keywords.kw_mark);
next(); // Consume "mark".
next(); // Consume first token (so we fix leading whitespace).
----------------
Please fix (remove) this comment that consumes not only mark.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D125961/new/
https://reviews.llvm.org/D125961
More information about the cfe-commits
mailing list