[clang] [clang-format] Add SpacesInParensOption for attributes and filtering for repeated parens (PR #77522)

[Alibek Omarov via cfe-commits]

a1batross wrote:

Any news on this?

https://github.com/llvm/llvm-project/pull/77522
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-format] Add SpacesInParensOption for attributes and filtering for repeated parens (PR #77522)

[Alibek Omarov via cfe-commits]

a1batross wrote:

I have built clang-format with these changes. On current commit, it sadly 
doesn't remove the space in these patterns:
```c
memset( g_used, 0, sizeof( g_used ) );
if( FBitSet( ent->curstate.effects, EF_INVLIGHT ) )
```

After reverting last commit, it got somewhat closer:
```c
memset(g_used, 0, sizeof( g_used ));
if(FBitSet( ent->curstate.effects, EF_INVLIGHT ))
```

But is there a way to configure it like this?
```c
memset( g_used, 0, sizeof( g_used ))
if( FBitSet( ent->curstate.effects, EF_INVLIGHT ))
```

https://github.com/llvm/llvm-project/pull/77522
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-format] Add SpacesInParensOption for filtering repeated parens (PR #77522)

[Alibek Omarov via cfe-commits]

a1batross wrote:

> I saw some comments on this. Looks like some were deleted maybe. This is 
> still in my queue, but unclear when I'll get back to it.

Yeah, sorry, I posted a comment here but decided it was a bit irrelevant. :)

But going to ask anyway then. Can this setting remove the space from repeated 
parentheses pattern like this?

```c
foo( (void *)buf, size );
```

to:

```c
foo((void *)buf, size );
```

So it does remove it on one side where the C-style cast parentheses, but 
doesn't on the other side?

https://github.com/llvm/llvm-project/pull/77522
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits