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

examples: fix/silence -Wsign-conversion #13501

Closed
wants to merge 4 commits into from

Conversation

vszakats
Copy link
Member

@vszakats vszakats commented Apr 29, 2024

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
Copy link
Member Author

vszakats commented May 9, 2024

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.

None yet

1 participant