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.

Sign up for GitHub

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

Jump to bottom

modify C API internals to match C API definition in headers (fixes #92) #94

Merged
merged 1 commit into from
Apr 20, 2017
Merged

modify C API internals to match C API definition in headers (fixes #92) #94

merged 1 commit into from
Apr 20, 2017

Conversation

hobu
Copy link
Member

@hobu hobu commented Mar 3, 2017 •
edited
Loading

See #92 and #93 for discussion.

This PR implements fixes to API internals rather than changing the public API.

@hobu hobu mentioned this pull request Mar 3, 2017
Closed
@miklos1
Copy link
Contributor

miklos1 commented Mar 10, 2017

Is this going to be merged?

@hobu
Copy link
Member Author

hobu commented Mar 10, 2017

Yes, but I was waiting on verification from @rosshemsley as requested in #92

@miklos1
Copy link
Contributor

miklos1 commented Mar 10, 2017

Thanks for the information!

@miklos1
Copy link
Contributor

miklos1 commented Apr 20, 2017

Prod

@hobu hobu merged commit 4768bf3 into master Apr 20, 2017
@hobu
Copy link
Member Author

hobu commented Apr 20, 2017

@miklos1 did this cure issues for you?

@hobu hobu deleted the capi-fixes branch October 7, 2019 02:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hobu @miklos1