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

urldata: tweak the UserDefined struct #16097

Closed
wants to merge 1 commit into from
Closed

urldata: tweak the UserDefined struct #16097

wants to merge 1 commit into from

Conversation

bagder
Copy link
Member

@bagder bagder commented Jan 27, 2025

By better sticking to listing the struct members sorted by size, this struct is now 48 bytes smaller on my fairly maximized build, without removing anything.

Turned 'connect_only' into two bits instead of an unsigned char with two magic values.

Also put the 'gssapi_delegation' field within ifdef HAVE_GSSAPI.

By better sticking to listing the struct members sorted by size, this
struct is now 48 bytes smaller on my fairly maximized build, without
removing anything.

Turned 'connect_only' into two bits instead of an unsigned char with two
magic values.

Also put the 'gssapi_delegation' field within ifdef HAVE_GSSAPI.
@testclutch
Copy link

Analysis of PR #16097 at fcefc2f1:

Test 1191 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 35b1c15 Jan 27, 2025
@bagder bagder deleted the bagder/userdef branch January 27, 2025 14:47
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