Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

New intrinsic: set_action_return_value #8327

Merged
merged 10 commits into from Dec 19, 2019
Merged

New intrinsic: set_action_return_value #8327

merged 10 commits into from Dec 19, 2019

Conversation

heifner
Copy link
Contributor

@heifner heifner commented Dec 13, 2019 •
edited

Change Description

This PR adds support for the set_action_return_value intrinsic enabled by the ACTION_RETURN_VALUE consensus feature. set_action_return_value sets a value (packed blob char array) that is included in the action_receipt.return_value. Since it is part of action_receipt the value will be included in the action_receipt digest which is included in the block header action_mroot.

Consensus Changes

  • Consensus Changes

New consensus protocol feature ACTION_RETURN_VALUE.

API Changes

  • API Changes

state_history_plugin action_receipt_v1 added to include new action_receipt.return_value once ACTION_RETURN_VALUE is activated.

Documentation Additions

  • Documentation Additions

@heifner heifner added the CONSENSUS Introduces a change that may modify consensus protocol rules on an existing blockchain. label Dec 13, 2019
@heifner heifner mentioned this pull request Dec 13, 2019
Merged
2 tasks
@heifner heifner merged commit e6c276c into develop Dec 19, 2019
@heifner heifner deleted the action-return branch December 19, 2019 13:08
@heifner heifner mentioned this pull request Jan 15, 2020
Merged
3 tasks
@conr2d conr2d mentioned this pull request Jan 24, 2020
Closed
@theblockstalk theblockstalk mentioned this pull request Dec 20, 2020
Open
@djmbritt djmbritt mentioned this pull request Jan 5, 2023
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@larryk85 larryk85 larryk85 approved these changes

Assignees
No one assigned
Labels
CONSENSUS Introduces a change that may modify consensus protocol rules on an existing blockchain.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@heifner @larryk85