Skip to content

build: fix compiler warning with auths disabled #12227

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

Closed
wants to merge 3 commits into from

Conversation

vszakats
Copy link
Member

@vszakats vszakats commented Oct 29, 2023

./curl/lib/http.c:979:12: warning: unused function 'is_valid_auth_separator' [-Wunused-function]
static int is_valid_auth_separator(char ch)
           ^
5 warnings generated.

Follow-up to e92edfb #11490

Closes #12227

Verified

This commit was signed with the committer’s verified signature.
vszakats Viktor Szakats
```
./curl/lib/http.c:737:12: warning: unused variable 'result' [-Wunused-variable]
  CURLcode result = CURLE_OK;
           ^
./curl/lib/http.c:1143:12: warning: empty expression statement has no effect; remove unnecessary ';' to silence this warning [-Wextra-semi-stmt]
           ;
           ^
./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;
               ^
./curl/lib/http.c:979:12: warning: unused function 'is_valid_auth_separator' [-Wunused-function]
static int is_valid_auth_separator(char ch)
           ^
5 warnings generated.
```

Follow-up to 0d3956b curl#11895
Follow-up to e92edfb curl#11490

Closes #xxxxx
@github-actions github-actions bot added the HTTP label Oct 29, 2023
@vszakats

This comment was marked as outdated.

Verified

This commit was signed with the committer’s verified signature.
vszakats Viktor Szakats
@vszakats vszakats removed the on-hold label Oct 29, 2023

Verified

This commit was signed with the committer’s verified signature.
vszakats Viktor Szakats
@vszakats vszakats changed the title build: fix compiler warnings with auths disabled build: fix compiler warning with auths disabled Oct 29, 2023
@vszakats vszakats closed this in aca7d80 Oct 30, 2023
@vszakats vszakats deleted the warn-10 branch October 30, 2023 11:00
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.

None yet

1 participant