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

Remove 'conn->data' completely #6608

Closed
wants to merge 2 commits into from
Closed

Remove 'conn->data' completely #6608

wants to merge 2 commits into from

Conversation

@bagder
Copy link
Member

@bagder bagder commented Feb 15, 2021

This complete removes the struct Curl_easy *data struct member from the connectdata struct.

This PR contains the #6520 and #6522 commits at this point but they will be merged individually before this, so I'll remove them when that happens.

Wiki page about it: https://github.com/curl/curl/wiki/conn.data-considered-bad

@bagder bagder added the tidy-up label Feb 15, 2021
lib/http2.c Outdated Show resolved Hide resolved
bagder added 2 commits Feb 15, 2021
The Curl_easy pointer struct entry in connectdata is now gone.
@bagder bagder force-pushed the bagder/conn-data-is-gone branch from 2c28296 to 5c4d678 Feb 15, 2021
@bagder bagder closed this in cfff12a Feb 16, 2021
@bagder bagder deleted the bagder/conn-data-is-gone branch Feb 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant