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

Expand c++ stdlib support #12

Open
anderslanglands opened this issue Nov 29, 2020 · 2 comments
Open

Expand c++ stdlib support #12

anderslanglands opened this issue Nov 29, 2020 · 2 comments
Assignees
Labels
enhancement New feature or request
Projects

Comments

@anderslanglands
Copy link
Contributor

Currently there is basic support for std::vector by wrapping it as an opaquebytes type which can be passed around. This seems to work quite well, despite being quite unsafe. It's currently named cppmm_vector but this should probably be changed to std_vector for consistency. Support for destroying these objects also needs to be implemented (just needs a _drop function probably).

Once that's done we should add support for other stdlib types. So far I've only seen std::set in an API (in USD) but I don't doubt there are others out there as well.

@anderslanglands anderslanglands added the enhancement New feature or request label Nov 29, 2020
@anderslanglands anderslanglands self-assigned this Nov 29, 2020
@luke-titley
Copy link
Collaborator

Is it not possible for the stl types to be exposed in the same way that the wrapped library bindings will be exposed ?

@anderslanglands
Copy link
Contributor Author

That could potentially work, but it means adding stdlib to the list of supported libraries, rather than just special-casing certain types and handling them directly.

At this stage I'd much prefer to write dedicated support for the containers that we need than to try and make a general solution.

@scott-wilson scott-wilson added this to To do in Cppmm Apr 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Cppmm
To do
Development

No branches or pull requests

2 participants