HazardyKnusperkeks added inline comments.
================ Comment at: clang/lib/Format/UnwrappedLineFormatter.cpp:119-121 + if (!RootToken.Next) { + return true; + } ---------------- Drop Braces ================ Comment at: clang/lib/Format/UnwrappedLineFormatter.cpp:122-126 + auto COperatorMatch = std::lower_bound(COperatorsFollowingVar.begin(), + COperatorsFollowingVar.end(), + RootToken.Next->Tok.getKind()); + if ((COperatorMatch == COperatorsFollowingVar.end() || + *COperatorMatch != RootToken.Next->Tok.getKind()) && ---------------- Use `std::binary_search` instead of `std::lower_bound`. That should simplify the following `if`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117416/new/ https://reviews.llvm.org/D117416 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits