-
Notifications
You must be signed in to change notification settings - Fork 52
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
FFI API #738
FFI API #738
Conversation
im not sure why the docs fail but maybe its the empty links, but all other tests pass. |
Apologies been busy, I def will get round to this, thank you so much! Be good to get your eyes too if you have any spare time @briandowns 🙂 |
All good no stress |
.gitignore
Outdated
/Release | ||
examples/ffi-example/build |
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.
Can you add new line characters to any file that don't have them?
I love this. Very very cool. Few things:
I can't wait to get started using this! |
Uh if you can help me build the test libs with the runner sure. Since they are required for the tests.
Im not entirely sure what you mean by embedded here, as in including shared objects into a binary? Im a tad confused.
We can do that idm about the name at all. |
@@ -0,0 +1,643 @@ | |||
#ifndef dictumod_include_h |
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.
It's probably worth adding this to the include directory within src, that way people may find it easier rather than it being in the examples directory
This is really really awesome!!! Thank you so much for working on this!! I'm gonna merge this in but left a little comment about moving / copying one file that any of us can pick up at some point (not massively urgent) |
FFI Api for dictu.
What's Changed:
Adds a FFI api which allows a C abi to easy extend dictu with native functions.
dictu-include.h
) which contains all macros which might be useful for external users, although of course theres the chance i missed something.dictu_internal_ffi_init
), this avoids the user having to link against dictu or duplicate function definitions.Type of Change:
Housekeeping: