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

darwin: silence compiler warnings #1722

Closed
wants to merge 1 commit into from
Closed

Conversation

@bagder
Copy link
Member

@bagder bagder commented Aug 2, 2017

With a clang pragma and three type fixes

Previous attempt to silencing compiler warnings in darwinssl.c was found in #1705

With a clang pragma and three type fixes
@bagder bagder requested a review from nickzman Aug 2, 2017
bagder added a commit that referenced this pull request Aug 2, 2017
With a clang pragma and three type fixes

(this is cherry-picked from #1722 to see if the build works out fine)
@coveralls
Copy link

@coveralls coveralls commented Aug 2, 2017

Coverage Status

Coverage decreased (-0.05%) to 75.26% when pulling 38c9032 on bagder/darwinssl-warnings-pragma into 1767adf on master.

@bagder
Copy link
Member Author

@bagder bagder commented Aug 2, 2017

This wasn't complete - I'm working out the additional details in #1706

@bagder bagder closed this in af271ce Aug 3, 2017
@bagder bagder deleted the bagder/darwinssl-warnings-pragma branch Aug 3, 2017
@lock lock bot locked as resolved and limited conversation to collaborators Jan 18, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.