[PATCH] D137762: [clang-format] avoid breaking )( with BlockIndent

Gedare Bloom via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 17 11:31:44 PST 2022


gedare added inline comments.


================
Comment at: clang/lib/Format/TokenAnnotator.cpp:5007
       return false;
     }
     const FormatToken *Previous = Right.MatchingParen->Previous;
----------------
owenpan wrote:
> Can we simply add the above and leave the original `return` statement on line 5009 unchanged?
Yes this seems to work. I'll push a diff shortly along with reverting the EXPECT_EQ outside of my new tests.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D137762/new/

https://reviews.llvm.org/D137762



More information about the cfe-commits mailing list