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
Add Pointer#read_array_of and Pointer#write_array_of #526
base: master
Are you sure you want to change the base?
Conversation
@tduehr, anything wrong with this ? |
@tduehr, can you either close this with a good reason or merge ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This needs tests for structs. Both going to and coming from C. Also needs tests for maximum values for an integer type.
c7afdbd
to
d644c08
Compare
@tduehr, ok, added some more specs and reimplemented straight in C; hope you deem this merge-worthy now. |
Needs a bit of tweaking, but I'll get it from here. |
…of, Pointer#put_array_of
d644c08
to
0156293
Compare
@tduehr, refactored the code, for consistency it now raises |
@tduehr any chance of getting this in ? |
This PR deprecates
Pointer#read_array_of_type
andPointer#write_array_of_type
and introducesPointer#read_array_of
andPointer#writer_array_of
.The former two weren't useful at all as one had to pass, along with the type, a corresponding writer/reader.