Skip to content

CURLOPT: bump CURL_REDIR_* macros to long#18110

Closed
vszakats wants to merge 1 commit intocurl:masterfrom
vszakats:opt-redir-long
Closed

CURLOPT: bump CURL_REDIR_* macros to long#18110
vszakats wants to merge 1 commit intocurl:masterfrom
vszakats:opt-redir-long

Conversation

@vszakats
Copy link
Member

This patch bumps the size of these macros from int to long, while
keeping their actual values the same. It may cause incompatibilities in
user code, requiring the bump of holder variables and/or adding casts:

  • CURL_REDIR_GET_ALL
  • CURL_REDIR_POST_301
  • CURL_REDIR_POST_302
  • CURL_REDIR_POST_303
  • CURL_REDIR_POST_ALL

Also:

  • keep existing cast within the documentation to make sure it applies
    to older curl versions as well.

This patch bumps the size of these macros from `int` to `long`, while
keeping their actual values the same. It may cause incompatibilities in
user code, requiring the bump of holder variables and/or adding casts:

- CURL_REDIR_GET_ALL
- CURL_REDIR_POST_301
- CURL_REDIR_POST_302
- CURL_REDIR_POST_303
- CURL_REDIR_POST_ALL

Also:
- keep existing cast within the documentation to make sure it applies
  to older curl versions as well.
@vszakats vszakats marked this pull request as draft July 30, 2025 21:57
@github-actions github-actions bot added the tests label Jul 30, 2025
@vszakats vszakats marked this pull request as ready for review July 30, 2025 22:17
@vszakats vszakats closed this in e5cf622 Jul 31, 2025
@vszakats vszakats deleted the opt-redir-long branch July 31, 2025 07:35
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.

1 participant

Comments