-
Notifications
You must be signed in to change notification settings - Fork 743
remove_prefix missing #428
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
Comments
This seems like a really hacky thing to do. I think there are better ways to do this, mainly re-exporting structs with a different name. That may be some churn, though, and it's definitely something we could automate... |
@emilio are you still planning on working on this? It's one of the last pieces preventing me from upgrading from 0.19. |
I... Don't really know how this would work for C++, but it shouldn't be hard to do for C. I don't have a particular incentive to work on this though I could give it a shot when I have some more free time. In any case, the place to apply the option would be here: That already has access to |
What do you think about adding another function on |
Hmm, I'd rather not expose the |
It may be somewhat tricky to make it work correctly though... |
I opened a pull request that adds an item_name callback. It is sufficient for our needs in rust-mbedtls. Providing the item kind could be useful but |
Add item_name parse callback. Issue: #428
Hi emilio, would it be possible to make a new release on crates.io? |
Sure, done :) |
Thanks emilio, is it expected I don't see it on crates.io? |
Blergh, no, was not expected. I recently switched distro version, and I didn't keep the cargo api token around, so the publishing failed. Should be in crates.io now. |
Thanks! The crate update is available as expected. |
I already mentioned this in #21. I was using remove_prefix in bindgen 0.19. This means I can't upgrade to 0.20 without introducing an API change in my -sys crate.
The text was updated successfully, but these errors were encountered: