Skip to content

chore: split JNI logic from pure golang logic#260

Merged
garyschulte merged 7 commits intobesu-eth:mainfrom
kevaundray:kw/split-pure-go-methods
Mar 19, 2025
Merged

chore: split JNI logic from pure golang logic#260
garyschulte merged 7 commits intobesu-eth:mainfrom
kevaundray:kw/split-pure-go-methods

Conversation

@kevaundray
Copy link
Copy Markdown
Contributor

@kevaundray kevaundray commented Mar 17, 2025

This is built ontop of #259 so the diff will include those changes as that PR is still open.

This essentially splits the logic into three parts:

  • Validate input and convert to golang data structures (this part is "unsafe" due to interactions with JNI and C)
  • Process the input and perform the necessary operation (this will be a golang slice)
  • Copy the output into the output buffer (this part is also "unsafe" if the output buffer does not satisfy the JNI requirements)

@garyschulte
Copy link
Copy Markdown
Contributor

can you squash commit/rebase to fix the DCO requirement?

kevaundray and others added 5 commits March 17, 2025 15:41
Signed-off-by: Kevaundray Wedderburn <kevtheappdev@gmail.com>
Signed-off-by: Daniel Lehrner <daniel.lehrner@consensys.net>
Signed-off-by: Kevaundray Wedderburn <kevtheappdev@gmail.com>
Signed-off-by: garyschulte <garyschulte@gmail.com>
Signed-off-by: Kevaundray Wedderburn <kevtheappdev@gmail.com>
Signed-off-by: Kevaundray Wedderburn <kevtheappdev@gmail.com>
Signed-off-by: garyschulte <garyschulte@gmail.com>
Signed-off-by: Kevaundray Wedderburn <kevtheappdev@gmail.com>
@kevaundray kevaundray force-pushed the kw/split-pure-go-methods branch from cb91462 to 73c84f8 Compare March 17, 2025 15:41
@kevaundray kevaundray marked this pull request as ready for review March 17, 2025 15:43
@kevaundray
Copy link
Copy Markdown
Contributor Author

can you squash commit/rebase to fix the DCO requirement?

Should be fixed now :)

Signed-off-by: kevaundray <kevtheappdev@gmail.com>
@garyschulte garyschulte enabled auto-merge (squash) March 19, 2025 16:26
@garyschulte garyschulte merged commit d5be37f into besu-eth:main Mar 19, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants