Skip to content

remove braces while returning #8007

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

Closed
wants to merge 1 commit into from
Closed

Conversation

anytizer
Copy link
Contributor

return(...); => return ...;
Not necessary to enclose the returning function call within braces,

return(...); => return ...;
@messmerd
Copy link
Member

There are probably a hundred instances of this throughout the codebase besides just this one return statement. I think cosmetic PRs like this that make no functional or structural changes should offer more to justify being merged.

@allejok96
Copy link
Contributor

If you edit a line in a commit, you should fix its formatting. But changing random lines to fix code formatting will only make it harder to git blame and browse git history. If we were to do such a code format fix it should be done all at once across the entire code base.

@AW1534 AW1534 closed this Jul 14, 2025
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

Successfully merging this pull request may close these issues.

4 participants