-
Notifications
You must be signed in to change notification settings - Fork 157
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
StructRef inner class. #85
Conversation
…rect memory and writes it's address to Struct memory.
depends on #82 |
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.
Looks ok to me. Seems to logically mirror the other Ref type fields. Would like @henriquemotaesteves and anyone else actively working on jnr-ffi to have a look too.
@headius already done that |
@goto1134 Oh of course. I have no remaining reservations. |
@goto1134 I'd like to get at least one other person to weigh in. Are you opposed to pushing this to a 2.1.x release to give it a bit more review time? I'd like to get 2.1 out today. |
@headius that's ok |
@henriquemotaesteves I'll probably go ahead and merge this soon unless you have concerns. |
This is too scary to do for the JRuby 9.1.6.0 release today, but I'll merge it afterwards and do a snapshot for testing. |
It addresses to a Struct or array of Structs.