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

No option for brace breaking after C# getter/setter #61968

Open
StrangeRanger opened this issue Apr 6, 2023 · 3 comments
Open

No option for brace breaking after C# getter/setter #61968

StrangeRanger opened this issue Apr 6, 2023 · 3 comments
Assignees
Labels
awaiting-review Has pending Phabricator review c# clang-format

Comments

@StrangeRanger
Copy link

It's possible that I didn't try all the possible options, but it doesn't seem like there is a suitable brace break option that allows you to break before braces after a getter/setter. For example, it changes the following code:

get
{
.....
}

into

get {
......
}

when I want it to remain as

get
{
.....
}
@StrangeRanger StrangeRanger changed the title [Clang-Format] No option for brace breaking after C# getter [Clang-Format] No option for brace breaking after C# getter/setter Apr 6, 2023
@llvmbot
Copy link
Collaborator

llvmbot commented Apr 6, 2023

@llvm/issue-subscribers-clang-format

@StrangeRanger StrangeRanger changed the title [Clang-Format] No option for brace breaking after C# getter/setter No option for brace breaking after C# getter/setter Apr 6, 2023
@mydeveloperday mydeveloperday self-assigned this Apr 16, 2023
@mydeveloperday
Copy link
Contributor

@StrangeRanger could you comment on https://reviews.llvm.org/D148467 in particular how we should handle

string Foo { set;get; }

@mydeveloperday mydeveloperday added the awaiting-review Has pending Phabricator review label Apr 16, 2023
@StrangeRanger
Copy link
Author

StrangeRanger commented Apr 18, 2023

@mydeveloperday Yes, I can do that in a bit. Sorry for the late reply.

Edit: Alright, I've commented at the link.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting-review Has pending Phabricator review c# clang-format
Projects
None yet
Development

No branches or pull requests

4 participants