Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

clang-format: penalty for breaking double brackets for c++ attributes #189

Closed
mdavis199 opened this issue Mar 19, 2020 · 1 comment
Closed

Comments

@mdavis199
Copy link

There should be a penalty setting that prevents breaking the double-brackets around c++ attributes.

Using the attached clang-format, the following code:

int PluginColumnModel::columnCount([[maybe_unused]] const QModelIndex& parent) const {}

is formated as

int PluginColumnModel::columnCount([
    [maybe_unused]] const QModelIndex& parent) const {}

clang-format.txt

mmitche pushed a commit to mmitche/llvm-project that referenced this issue Aug 4, 2022
…406.10 (llvm#189)

[objwriter/12.x] Update dependencies from dotnet/arcade
@rymiel
Copy link
Member

rymiel commented Sep 21, 2022

Fixed by 5a4ddbd

There probably doesn't need to be a config option for this because I'd consider the two square brackets ever being broken up as a bug

@rymiel rymiel closed this as completed Sep 21, 2022
vext01 pushed a commit to vext01/llvm-project that referenced this issue Aug 13, 2024
Remove trace inputs struct passed into the control point.
mjklemm added a commit to mjklemm/llvm-project that referenced this issue Oct 19, 2024
* Remove tests for OpenACC

* Modify mixed OpenMP/OpenACC tests to no have OpenACC

* Delete CUDA Fortran test
ergawy pushed a commit to ergawy/llvm-project that referenced this issue Oct 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants