Skip to content
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

WIP Added first working version of ABI generation for functions #8

Closed
wants to merge 1 commit into from

Conversation

maoueh
Copy link
Collaborator

@maoueh maoueh commented Aug 27, 2022

It's possible to:

  • Transform a pb::eth::v2::Call into ABI generated "function" (extracts decoded params of a call)
  • Decode the output of pb::eth::v2::Call (extracts decoded output params of a call)
  • Encode ABI generated "function" into Ethereum packed binary encoding
  • Perform RPC call for any function defined in the ABI with automatic encoding of input and decoding of output

There is good chance the API will slightly change or be improved based on feedback.

It's possible to:
- Transform a `pb::eth::v2::Call` into ABI generated "function" (extracts decoded params of a call)
- Decode the output of `pb::eth::v2::Call` (extracts decoded output params of a call)
- Encode ABI generated "function" into Ethereum packed binary encoding
- Perform RPC call for any function defined in the ABI with automatic encoding of input and decoding of output
@maoueh maoueh marked this pull request as draft August 27, 2022 05:30
ParamType::Int(_) => {
quote! {
{
let mut v = [0 as u8; 32];
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great catch >.<

@maoueh
Copy link
Collaborator Author

maoueh commented Aug 31, 2022

Merged in develop already

@maoueh maoueh closed this Aug 31, 2022
@maoueh maoueh deleted the feature/functions-encode-decode-call branch August 31, 2022 15:27
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.

None yet

2 participants