-
Notifications
You must be signed in to change notification settings - Fork 59
Update c bindings, add new functions #50
Update c bindings, add new functions #50
Conversation
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.
Update main.c
to cover the new functions.
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.
LGTM!
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 good, but move impl of address_from_string
into main crate?
use iota::client::bytes_to_trytes; | ||
|
||
#[no_mangle] | ||
pub extern "C" fn address_from_string(c_addr: *const c_char) -> *const Address { |
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 is a nice function in general, it's going to be used in other bindings. Maybe move the impl to the main crate?
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.
Yeah I think that's a good idea, i'll do that before push
Signed-off-by: DyrellC <dyrell.chapman@iota.org>
Description of change
Updating c bindings to include tagged and signed packets on the subscriber end, as well as a couple utility functions for usability (like converting string link representation to proper streams Address object through bindings).
Type of change
How the change has been tested
I have run each function through modifications of the main.c file, and confirmed they work when pulled into golang
Change checklist