crayroud added inline comments.
================
Comment at: clang/lib/Format/TokenAnnotator.cpp:3469
return true;
- if (Right.is(TT_OverloadedOperatorLParen))
- return spaceRequiredBeforeParens(Right);
- if (Left.is(tok::comma))
+ if (Left.is(tok::comma) && !Right.is(TT_OverloadedOperatorLParen))
return true;
----------------
MyDeveloperDay wrote:
> I'm ever so slightly struggling to see where this case is covered. Could you
> give me the line number?
Left.is(tok::comma) is used to always add a space after a coma, but we want to
be able to configure the space after the coma in the following example:
```
bool operator,();
```
Verified by:
```
25201: verifyFormat("bool operator,();");
```
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D110833/new/
https://reviews.llvm.org/D110833
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
