Skip to content

New EEI method: abort #43

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Closed
axic opened this issue Aug 14, 2016 · 4 comments
Closed

New EEI method: abort #43

axic opened this issue Aug 14, 2016 · 4 comments
Labels
EEI (environment interface) enhancement

Comments

@axic
Copy link
Member

axic commented Aug 14, 2016

Abort execution and store a reason.

Parameters

  • reasonCode i32 the reason code
  • descriptionOffset i32 the memory offset to load the reason text from
  • descriptionLength i32 the length of the reason text (limited to 32 bytes)

Returns

nothing
@axic
Copy link
Member Author

axic commented Aug 14, 2016 •
edited
Loading

Related ethereum/EIPs#62. It could also be replaced with special LOGs, which are not rolled back at execution errors. See ethereum/EIPs#120 for an implementation of moving LOGs to storage.

@wanderer
Copy link
Member

why not just use return?

@axic
Copy link
Member Author

axic commented Aug 18, 2016

Assuming we change return so that the value is not rolled back on a VM error that could work. However that introduces a discrepancy with how EVM expects it.

@nmushegian nmushegian mentioned this issue Aug 22, 2016
Closed
@axic
Copy link
Member Author

axic commented Jan 17, 2018

Solved by https://github.com/ethereum/EIPs/blob/master/EIPS/eip-140.md

@axic axic closed this as completed Jan 17, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
EEI (environment interface) enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@axic @wanderer