Skip to content

lib: use Curl_str_number() for parsing decimal numbers#16319

Closed
bagder wants to merge 6 commits intomasterfrom
bagder/strtoul-replace
Closed

lib: use Curl_str_number() for parsing decimal numbers#16319
bagder wants to merge 6 commits intomasterfrom
bagder/strtoul-replace

Conversation

@bagder
Copy link
Copy Markdown
Member

@bagder bagder commented Feb 13, 2025

Instead of strtoul() and strtol() calls.

Easier API with better integer overflow detection and built-in max check that now comes automatic everywhere this is used.

@github-actions github-actions bot added the tests label Feb 13, 2025
bagder added a commit that referenced this pull request Feb 13, 2025
Instead of strtoul() and strtol() calls.

Easier API with better integer overflow detection and built-in max check
that now comes automatic everywhere this is used.

Closes #16319
bagder added a commit that referenced this pull request Feb 13, 2025
Since the former function is getting removed from the lib.

Closes #16319
@bagder bagder force-pushed the bagder/strtoul-replace branch from 7812998 to 8fb47d2 Compare February 13, 2025 14:33
bagder added a commit that referenced this pull request Feb 13, 2025
bagder added a commit that referenced this pull request Feb 14, 2025
Instead of strtoul() and strtol() calls.

Easier API with better integer overflow detection and built-in max check
that now comes automatic everywhere this is used.

Closes #16319
bagder added a commit that referenced this pull request Feb 14, 2025
Since the former function is getting removed from the lib.

Closes #16319
bagder added a commit that referenced this pull request Feb 14, 2025
@bagder bagder force-pushed the bagder/strtoul-replace branch from 8fb47d2 to 177a6af Compare February 14, 2025 07:47
Instead of strtoul() and strtol() calls.

Easier API with better integer overflow detection and built-in max check
that now comes automatic everywhere this is used.

Closes #16319
Since the former function is getting removed from the lib.

Closes #16319
@bagder bagder force-pushed the bagder/strtoul-replace branch from 8eb2bdb to d8f8e3a Compare February 14, 2025 09:07
@bagder bagder closed this in 130b689 Feb 14, 2025
bagder added a commit that referenced this pull request Feb 14, 2025
Instead of strtoul() and strtol() calls.

Easier API with better integer overflow detection and built-in max check
that now comes automatic everywhere this is used.

Closes #16319
bagder added a commit that referenced this pull request Feb 14, 2025
Since the former function is getting removed from the lib.

Closes #16319
bagder added a commit that referenced this pull request Feb 14, 2025
@bagder bagder deleted the bagder/strtoul-replace branch February 26, 2025 07:48
pps83 pushed a commit to pps83/curl that referenced this pull request Apr 26, 2025
pps83 pushed a commit to pps83/curl that referenced this pull request Apr 26, 2025
Instead of strtoul() and strtol() calls.

Easier API with better integer overflow detection and built-in max check
that now comes automatic everywhere this is used.

Closes curl#16319
pps83 pushed a commit to pps83/curl that referenced this pull request Apr 26, 2025
Since the former function is getting removed from the lib.

Closes curl#16319
pps83 pushed a commit to pps83/curl that referenced this pull request Apr 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

1 participant