-
Notifications
You must be signed in to change notification settings - Fork 153
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
Can't tell autocxx that the replaced type is POD and safe to use in other PODs #1192
Comments
Thanks. I agree this is seems like a valid bug. It would be even better if you could raise a PR with a failing test case in |
I'll try to get to it, but have a bit much to unwrap locally at the moment. |
Landed a fix in a branch, but it's made me realize this bit of the autocxx codebase is a bit... legacy. I might do some more thinking about it over the next week or so in case I think it merits a more substantial rewrite. |
This shrinks us down to enumerating the fields in a struct in one place instead of two. However, it adds a lot of code, so possibly not worth it. Relates to #1192.
Hi, just commenting to say I'm interested in this use case, coming from the linear algebra and c++ world. |
Expected Behavior
I should be able to replace a C++ type autocxx doesn't understand with my custom Rust type, and tell autocxx that it's a POD so it can be used in
generate_pod!
for other types like this:(Eigen is a header-only library, so should be relatively easy to add for a repro, but I suppose this can be reproduced with simpler custom types too)
Actual Behavior
autocxx complains that
MyStruct
can't be generated as a POD because the dependency type "isn't known":Specifications
The text was updated successfully, but these errors were encountered: