Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
clang/lib/Format/ContinuationIndenter.cpp | ||
---|---|---|
1290–1293 | Unrelated to your change, it only shed light on this: Shouldn't it be this? |
clang/lib/Format/ContinuationIndenter.cpp | ||
---|---|---|
1290–1293 | Yeah, I will update it. |
clang/lib/Format/ContinuationIndenter.cpp | ||
---|---|---|
1290–1293 | This is in my view one of the reasons why I personally like it when we comment the condition with an example and don't have these huge compound expressions. Often they are covering corner cases. (we should call them out) |
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 TokenTypes (?), for instance, both TT_BinaryOperator and TT_DictLiteral?
Shouldn't it be this?