-
Notifications
You must be signed in to change notification settings - Fork 2.4k
FFI compatible token interface #48
FFI compatible token interface #48
Conversation
Our own Why add the auto-generated code to git? |
@garious I'll give our own options a shot, probably want that to live in our SDK eventually. Checking in the header has two advantages, non-rust projects that want to pull use the header don't have to use Rust to generate it. Also its more explicit when someone intentionally or unintentionally makes a change that affects the public interface. |
99a35c4
to
8ae1535
Compare
@garious I added a repr(C) version of Option, was this the route you had in mind? |
58e8363
to
4d35ecc
Compare
Will move option.rs to the SDK once/if approved |
4d35ecc
to
2c2c292
Compare
@garious What do you think of our use of own |
@jackcmay, no objection. Buy maybe name it |
…na-1.7.12 upgrade to solana 1.7.12
Token account definitions were not FFI compatible because of the use of
Option<T>