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

escape: use table lookup when adding %-codes to output #10377

wants to merge 1 commit into from


Copy link

@bagder bagder commented Jan 31, 2023

Faster URL encodes in libcurl.

I wrote a test program that uses a 1MB buffer in which byte sequences are stored from 0 to 255 over and over.

Then curl_easy_escape() is called repeatedly to generate a URL-encoded version of that input. N number of times.

On my dev host, the new code runs 7.8 times faster.

On my dev host, this code runs 7.8 times faster.

Closes #10377
@bagder bagder closed this in fc8ad0b Jan 31, 2023
@bagder bagder deleted the bagder/escape-table branch January 31, 2023 21:43
bch pushed a commit to bch/curl that referenced this pull request Jul 19, 2023
On my dev host, this code runs 7.8 times faster.

Closes curl#10377
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Successfully merging this pull request may close these issues.

None yet

1 participant