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: remove values that are not used depending on compilation settings #13188
Conversation
STRING_SSL_CRLFILE, /* crl file to check certificate */ | ||
STRING_SSL_ISSUERCERT, /* issuer cert file to check certificate */ | ||
STRING_SERVICE_NAME, /* Service name */ | ||
#ifndef CURL_DISABLE_PROXY |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is in direct contradiction to the comment above the enum which says:
This enumeration MUST not use conditional directives (#ifdefs)
But: the comment also refers to a (misspelled) source code file that was removed in 821d108, which makes me suspect maybe this comment does not apply anymore?
@monnerat can you fill in some blanks maybe?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems the comment is obsolete: the current test is now based on CURLOPT_*
, not on STRING_*
anymore.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@MAntoniak can you perhaps update that comment to reflect reality?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If I understand correctly just remove it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
…tings. Reduced size of dynamically_allocated_data structure. Reduced number of stored values in enum dupstring and enum dupblob. This affects the reduced array placed in the UserDefined structure.
Thanks! |
Reduced size of dynamically_allocated_data structure. Reduced number of stored values in enum dupstring and enum dupblob. This affects the reduced array placed in the UserDefined structure. Closes curl#13188
Reduced size of dynamically_allocated_data structure.
Reduced number of stored values in enum dupstring and enum dupblob. This affects the reduced array placed in the UserDefined structure.
It is also possible to remove the STRING_SASL_AUTHZID and STRING_BEARER values from the dupstring. However, this requires a separate PR.