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.

Sign up for GitHub

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

Jump to bottom

[clang-format] PackConstructorInitializers only on next line #60241

Closed
AdamBucior opened this issue Jan 23, 2023 · 1 comment
Closed

[clang-format] PackConstructorInitializers only on next line #60241

AdamBucior opened this issue Jan 23, 2023 · 1 comment
Assignees
@Backl1ght
Labels
clang-format

Comments

@AdamBucior
Copy link

AdamBucior commented Jan 23, 2023 •
edited by VoltrexKeyva

Hi,

Would it be possible to add an option to PackConstructorInitializers similar to NextLine but which doesn't try to put the initializers on the current line?

Example:

Constructor()
    : a(), b()

Constructor()
    : aaaaaaaaaaaaaaaaaaaa(), bbbbbbbbbbbbbbbbbbbb(), ddddddddddddd()

Constructor()
    : aaaaaaaaaaaaaaaaaaaa(),
      bbbbbbbbbbbbbbbbbbbb(),
      cccccccccccccccccccc()
@llvmbot
Copy link
Collaborator

llvmbot commented Jan 23, 2023

@llvm/issue-subscribers-clang-format

@Backl1ght Backl1ght self-assigned this Feb 1, 2023
CarlosAlbertoEnciso pushed a commit to SNSystems/llvm-debuginfo-analyzer that referenced this issue Feb 7, 2023
@Backl1ght @CarlosAlbertoEnciso
[clang-format] PackConstructorInitializers support PCIS_OnlyNextLine
1a63a93 
fixes llvm/llvm-project#60241

Differential Revision: https://reviews.llvm.org/D143091
skatrak pushed a commit to skatrak/llvm-project-rocm that referenced this issue Feb 10, 2023
@Backl1ght @skatrak
[clang-format] PackConstructorInitializers support PCIS_OnlyNextLine
f080dbd 
fixes llvm/llvm-project#60241

Differential Revision: https://reviews.llvm.org/D143091
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Backl1ght Backl1ght

Labels
clang-format
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@EugeneZelenko @AdamBucior @Backl1ght @llvmbot