Skip to content

examples: fix/silence -Wsign-conversion - #13501

Closed
vszakats wants to merge 4 commits into
curl:masterfrom
vszakats:signwarn-07
Closed

examples: fix/silence -Wsign-conversion#13501
vszakats wants to merge 4 commits into
curl:masterfrom
vszakats:signwarn-07

Conversation

@vszakats

@vszakats vszakats commented Apr 29, 2024

Copy link
Copy Markdown
Member

Cherry-picked from #13489
Follow-up to 3829759 #12489
Closes #13501

@vszakats vszakats changed the title examples: silence -Wsign-conversion examples: fix/silence -Wsign-conversion May 9, 2024
@vszakats

vszakats commented May 9, 2024

Copy link
Copy Markdown
Member Author

If no objections I'd go along merging this.

It has legitimate fixes and there is only one sign cast.

This one, if undesired, can be deleted or beefed up with sign checks. It's used
when converting the return value of if_nametoindex (int or long unsigned
depending on platform) to the long needed by curl_easy_setopt().

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.

1 participant