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

lib, src, tests: added space around ternary expressions #14912

Closed
wants to merge 1 commit into from

Conversation

gblmrn
Copy link
Contributor

@gblmrn gblmrn commented Sep 14, 2024

Added space around ternary expressions to be in accordance with code style

src/tool_getparam.c Dismissed Show dismissed Hide dismissed
@bagder
Copy link
Member

bagder commented Sep 15, 2024

I think this is fine, although this PR also leaves out a lot of other instances using the same pattern.

It made me add more checks to checksrc for this, which made me find many other nits but I'll do them in a separate PR.

@gblmrn
Copy link
Contributor Author

gblmrn commented Sep 15, 2024

Sure, glad to help.

@bagder bagder closed this in 5a26371 Sep 18, 2024
@bagder
Copy link
Member

bagder commented Sep 18, 2024

Thanks!

@gblmrn
Copy link
Contributor Author

gblmrn commented Sep 18, 2024

Anytime

moritzbuhl pushed a commit to moritzbuhl/curl that referenced this pull request Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants