-
Notifications
You must be signed in to change notification settings - Fork 35
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
chore: Switch to ffi_interface
#139
chore: Switch to ffi_interface
#139
Conversation
c7bfd13
to
57e6879
Compare
Depends on #142 |
08eb67f
to
d6e0ba5
Compare
Signed-off-by: Kevaundray Wedderburn <kevtheappdev@gmail.com>
Signed-off-by: Kevaundray Wedderburn <kevtheappdev@gmail.com>
Signed-off-by: Kevaundray Wedderburn <kevtheappdev@gmail.com>
Signed-off-by: Kevaundray Wedderburn <kevtheappdev@gmail.com>
Signed-off-by: Kevaundray Wedderburn <kevtheappdev@gmail.com>
Signed-off-by: Kevaundray Wedderburn <kevtheappdev@gmail.com>
d6e0ba5
to
c950cc7
Compare
@@ -27,7 +27,7 @@ jobs: | |||
export PATH=$GOROOT/bin:$PATH | |||
# rust dependencies | |||
export CARGO_HOME="$HOME/.cargo" | |||
curl https://sh.rustup.rs -sSf | sh -s -- -y --default-toolchain 1.68.2 | |||
curl https://sh.rustup.rs -sSf | sh -s -- -y --default-toolchain 1.75.0 |
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 change was added because CI uses stable and matt was getting an unstable error from one of the newer libraries for a feature which should be stable as of 1.75. (1.68 is quite old)
Signed-off-by: Kevaundray Wedderburn <kevtheappdev@gmail.com>
public void testCallLibraryWithMaxElements() { | ||
Bytes32 element = Bytes32.fromHexString("0xd36f20567f74f607d9252186ff8efed04de4578d1ddb3de4fe6c5e4249e0045b"); | ||
Bytes32 element = Bytes32.fromHexString("0x006f20567f74f607d9252186ff8efed04de4578d1ddb3de4fe6c5e4249e0045b"); | ||
Bytes32[] arr = new Bytes32[256]; | ||
for (int i = 0; i < 256; i++) { | ||
arr[i] = element; | ||
} |
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.
These changes are from #142
…e-to-ffi-interface Signed-off-by: Karim Taam <karim.t2am@gmail.com> # Conflicts: # ipa-multipoint/src/test/resources/genesis_lvl1_commits.json
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
This switches to the new
ffi_interface
folder and avoids any interface changes by using a global static.The global static should be replaced with a pointer as is done in #138