Skip to content

CURLOPT: bump remaining macros to long#18134

Closed
vszakats wants to merge 1 commit intocurl:masterfrom
vszakats:curloptmore
Closed

CURLOPT: bump remaining macros to long#18134
vszakats wants to merge 1 commit intocurl:masterfrom
vszakats:curloptmore

Conversation

@vszakats
Copy link
Copy Markdown
Member

@vszakats vszakats commented Aug 1, 2025

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:

  • CURLGSSAPI_DELEGATION_FLAG
  • CURLGSSAPI_DELEGATION_NONE
  • CURLGSSAPI_DELEGATION_POLICY_FLAG
  • CURLMIMEOPT_FORMESCAPE
  • CURLSSH_AUTH_AGENT
  • CURLSSH_AUTH_ANY
  • CURLSSH_AUTH_DEFAULT
  • CURLSSH_AUTH_GSSAPI
  • CURLSSH_AUTH_HOST
  • CURLSSH_AUTH_KEYBOARD
  • CURLSSH_AUTH_NONE
  • CURLSSH_AUTH_PASSWORD
  • CURLSSH_AUTH_PUBLICKEY

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:

- CURLGSSAPI_DELEGATION_FLAG
- CURLGSSAPI_DELEGATION_NONE
- CURLGSSAPI_DELEGATION_POLICY_FLAG

- CURLMIMEOPT_FORMESCAPE

- CURLSSH_AUTH_AGENT
- CURLSSH_AUTH_ANY
- CURLSSH_AUTH_DEFAULT
- CURLSSH_AUTH_GSSAPI
- CURLSSH_AUTH_HOST
- CURLSSH_AUTH_KEYBOARD
- CURLSSH_AUTH_NONE
- CURLSSH_AUTH_PASSWORD
- CURLSSH_AUTH_PUBLICKEY

Also:
- keep existing cast within the documentation to make sure it applies
  to older curl versions as well.
@vszakats vszakats changed the title CURLOPT: bump more (rare) macros to long CURLOPT: bump remaining macros to long Aug 1, 2025
@vszakats vszakats closed this in 80297e1 Aug 1, 2025
@vszakats vszakats deleted the curloptmore branch August 1, 2025 16:51
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