-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Initial _ctypes implementation #5519
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
Conversation
Signed-off-by: Ashwin Naren <arihant2math@gmail.com>
5d27774
to
aeb6d74
Compare
You don't need to close the previous PR to replace patch. You can push force to #5516 instead of opening a new one. |
Fair enough I'll keep them consistent. |
aeb6d74
to
d56e4f7
Compare
Adds sizeof and PyCSimple Signed-off-by: Ashwin Naren <arihant2math@gmail.com>
d56e4f7
to
fc7aa23
Compare
It passes now, should be good to review. |
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.
👍
Oh, could you add the code above to |
Signed-off-by: Ashwin Naren <arihant2math@gmail.com>
Done, but at some point this will be superseded by the actual |
Replacement of #5516.
The following code (adapted from the start of the
ctypes
module) runs: