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: use Curl_str_* instead of strtok_r() #16360

Closed
wants to merge 1 commit into from
Closed

Conversation

bagder
Copy link
Member

@bagder bagder commented Feb 17, 2025

Helps avoid extra mallocs. Gets rid of the private strtok_r implementation.

bagder added a commit that referenced this pull request Feb 17, 2025
Helps avoid extra mallocs. Gets rid of the private strtok_r
implementation.

Closes #16360
@bagder bagder marked this pull request as ready for review February 17, 2025 10:19
Helps avoid extra mallocs. Gets rid of the private strtok_r
implementation.

Closes #16360
@testclutch
Copy link

Analysis of PR #16360 at 24dd8f4d:

Test http/test_14_auth.py::TestAuth::test_14_03_digest_put_auth[h3] failed, which has NOT been flaky recently, so there could be a real issue in this PR.

Generated by Testclutch

@bagder bagder closed this in 676de7f Feb 17, 2025
@bagder bagder deleted the bagder/rm-strtok branch February 17, 2025 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants