Skip to content

Add section on C API #70

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

Merged
merged 1 commit into from
Nov 6, 2020
Merged

Add section on C API #70

merged 1 commit into from
Nov 6, 2020

Conversation

rgommers
Copy link
Member

@rgommers rgommers commented Nov 3, 2020

A C API is out of scope for the API standard, but there's quite some code out there using C/Cython APIs, and so recommendations on how to deal with that when library authors want to adopt this API standard are needed.

There's a lot that can be said here, more suggestions welcome. We can always add to this; it's a potentially very large topic, but it's also mostly a set of side notes on something that's out of scope. So I'm inclined to just get this in and adapt in response to more feedback/questions.

A C API is out of scope for the API standard, but there's quite some
code out there using C/Cython APIs, and so recommendations on how
to deal with that when library authors want to adopt this API standard
are needed.
Copy link
Contributor

@kgryte kgryte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@rgommers rgommers merged commit 7d96766 into master Nov 6, 2020
@rgommers rgommers deleted the c-api branch November 6, 2020 20:55
@rgommers
Copy link
Member Author

rgommers commented Nov 6, 2020

In it goes. Thanks for checking @kgryte

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants