Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

clang-format has no support for dangling parenthesis "( )" #146

Open
RadWolfie opened this issue Aug 2, 2021 · 1 comment
Open

clang-format has no support for dangling parenthesis "( )" #146

RadWolfie opened this issue Aug 2, 2021 · 1 comment
Labels
bug Something isn't working

Comments

@RadWolfie
Copy link
Member

This ticket will act as a watchlist for https://reviews.llvm.org/D33029 ticket.

How does this affect us?
Planned OOVPA_SIG_MATCH macro support to count all OV arrays on compile time will end up result to:

OOVPA_SIG_MATCH(
    OV_MATCH(0x04, 'S'),
    OV_MATCH(0x0B, 'E'),
    OV_MATCH(0x11, 'D'), );

Workaround solution is: (for time being)

OOVPA_SIG_MATCH(
    OV_MATCH(0x04, 'S'),
    OV_MATCH(0x0B, 'E'),
    OV_MATCH(0x11, 'D'),
    // add comment line here to force closing parenthesis stay on new line
);
@RadWolfie RadWolfie added the bug Something isn't working label Aug 2, 2021
@RadWolfie
Copy link
Member Author

Possible resolved in https://reviews.llvm.org/D109557 which will feature in clang 15.0 (not yet released)

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant