HazardyKnusperkeks added inline comments.
================
Comment at: clang/lib/Format/ContinuationIndenter.cpp:1294-1296
+ Previous->isOneOf(tok::l_paren, tok::comma, tok::colon,
TT_BinaryOperator,
+ TT_ConditionalExpr) &&
!Previous->isOneOf(TT_DictLiteral, TT_ObjCMethodExpr)) {
----------------
curdeius wrote:
> Unrelated to your change, it only shed light on this:
> This condition looks strange to me. How can `Previous` be at the same time
> two different `TokenType`s (?), for instance, both `TT_BinaryOperator` and
> `TT_DictLiteral`?
>
> Shouldn't it be this?
Yeah, I will update it.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D115069/new/
https://reviews.llvm.org/D115069
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
