rust-bindgen
rust-bindgen copied to clipboard
Added support for dynamic loading of variables
This expands support for dynamic loading to support dynamically loaded variables.
This PR addresses my own grievance - issue #2113. It plays nicely with --dynamic-link-require-all
. I tried to mirror the approach used to support dynamically loaded functions.
It's my first contribution to this project, all feedback welcome.
Warning
- These commits modify unsafe code. Please review it carefully!
Hey, thanks for looking at this.
Sure, I can try to add some tests. Would you rather have me add them here, or merge this and open a separate PR?
@emilio
Let's add them here, so the code doesn't land untested.
@emilio Sorry it took this long, I've added some tests. Let me know if there's anything outstanding still
:umbrella: The latest upstream changes (presumably 310f7f877042f0858cd01fb1c326a41d23bd821c) made this pull request unmergeable. Please resolve the merge conflicts.
@pradt2 do you have time to rebase this? I can do it if that's not the case
@pvdrz done
:umbrella: The latest upstream changes (presumably #2284) made this pull request unmergeable. Please resolve the merge conflicts.