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

starting braces after function() in new line #6

Open
amarts opened this issue May 3, 2018 · 2 comments
Open

starting braces after function() in new line #6

amarts opened this issue May 3, 2018 · 2 comments

Comments

@amarts
Copy link
Collaborator

amarts commented May 3, 2018

Noticed that with the current .clang-format the starting brace of the function is not in the new-line.

But if we use '-style=Mozilla' that specific thing is handled properly (not many other things. Not able to corner what is the exact option.

@nigelbabu @jdarcy @aravindavk @xhernandez

@xhernandez
Copy link
Contributor

In the last PR I sent, I changed BreakBeforeBraces from Attach to Linux. It caused all braces to be put into a new line after a function.

I see them correctly formatted in latest update of formatted-dht-common.c.

@xhernandez
Copy link
Contributor

I think this can be closed. Current formatting seems to work as expected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants