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

curl_ctype: convert to macros-only #9429

Closed
wants to merge 1 commit into from
Closed

Conversation

bagder
Copy link
Member

@bagder bagder commented Sep 5, 2022

This no longer provide functions, only macros. Runs slightly faster and produces smaller output.

@bagder
Copy link
Member Author

bagder commented Sep 5, 2022

I tested with and without this PR in my performance runs in #9408 and there is a fractional gain, both in speed and in output size.

This no longer provide functions, only macros. Runs faster and produces
smaller output.

The biggest precaution this change brings, is DO NOT use
post/pre-increments when passing arguments to the macros.

Closes #9429
@bagder bagder force-pushed the bagder/ctype-only-macros branch from 0f4fe7b to 67678d2 Compare Sep 5, 2022
@bagder bagder closed this in f65f750 Sep 6, 2022
@bagder bagder deleted the bagder/ctype-only-macros branch Sep 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant