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

Bring opaqueptr back online #38

Open
luke-titley opened this issue Feb 22, 2021 · 0 comments
Open

Bring opaqueptr back online #38

luke-titley opened this issue Feb 22, 2021 · 0 comments
Projects
Milestone

Comments

@luke-titley
Copy link
Collaborator

Currently all c wrapper records are implemented as an opaque block of bytes, this is not very portable.
The first version of cppmm had the option to choose opaqueptr which means the entire cpp record is allocated on the heap and the cpp record just passes a pointer around to that. This is far more portable, as the size and alignment of the cpp record do not affect the size and alignment of the c one.

@scott-wilson scott-wilson added this to To do in Cppmm Apr 24, 2021
@luke-titley luke-titley added this to the mvp milestone Apr 25, 2021
@luke-titley luke-titley moved this from To do to Reviewer approved in Cppmm May 4, 2021
@luke-titley luke-titley moved this from Reviewer approved to Done in Cppmm May 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Cppmm
Done
Development

No branches or pull requests

1 participant