Issue 138722
Summary Grouping Parentheses
Labels
Assignees
Reporter DarkTyger
    What do you think about adding `GroupingParentheses` to those options?

That is, `GroupingParentheses: false` would produce:

``` c++
    if( (x + 5) == 7 ) { … }
    int x = (((3 + y) * (j - 5)) + (k * k));
```

Currently, when given:

``` 
SpacesInCStyleCastParentheses: true
SpacesInConditionalStatement: true
SpacesInContainerLiterals: false
SpacesInParentheses: true
SpacesInParens: Custom
SpacesInParensOptions:
  ExceptDoubleParentheses: false
  InConditionalStatements: true
  InCStyleCasts: true
  InEmptyParentheses: false
  Other: true
```

clang-format generates the far-too-spacious (IMO):

``` c++
    if( ( x + 5 ) == 7 ) { … }
    int x = ( ( ( 3 + y ) * ( j - 5 ) ) + ( k * k ) );
```

_______________________________________________
llvm-bugs mailing list
llvm-bugs@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bugs

Reply via email to