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
compiler warnings in http.c when all auth methods disabled #12228
Labels
Comments
This was referenced Oct 29, 2023
Fixed in #12262 |
Reopening because of the unfixed warnings. |
Sorry, this is a more correct set of options to trigger the warnings:
This is on macOS. Also requires |
zuoxiaofeng
pushed a commit
to zuoxiaofeng/curl
that referenced
this issue
Nov 28, 2023
Fix compiler warnings in builds with disabled auths, NTLM and SPNEGO. E.g. with `CURL_DISABLE_BASIC_AUTH` + `CURL_DISABLE_BEARER_AUTH` + `CURL_DISABLE_DIGEST_AUTH` + `CURL_DISABLE_NEGOTIATE_AUTH` + `CURL_DISABLE_NTLM` on non-Windows. ``` ./curl/lib/http.c:737:12: warning: unused variable 'result' [-Wunused-variable] CURLcode result = CURLE_OK; ^ ./curl/lib/http.c:995:18: warning: variable 'availp' set but not used [-Wunused-but-set-variable] unsigned long *availp; ^ ./curl/lib/http.c:996:16: warning: variable 'authp' set but not used [-Wunused-but-set-variable] struct auth *authp; ^ ``` Regression from e92edfb curl#11490 Fixes curl#12228 Closes curl#12335
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I did this
Built non-Windows (e.g. Linux) curl with CMake options:
Ref:
0d3956b #11895
e92edfb #11490
This one fixed by: #12262
I expected the following
No compiler warnings.
curl/libcurl version
8.5.0-DEV master
operating system
any
The text was updated successfully, but these errors were encountered: