[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