Proto/apply: do not hash again in record_operation
Context
In apply.ml
, record_operation
hashes the operation, even though it is only called in apply_operation
, where we have access to the variable hash
which already contains the operation's hash (see apply_operation_with_mode
in main.ml
). This MR makes record_operation
take an additional argument hash
instead.
Checklist
- [N/A] Document the interface of any function added or modified (see the coding guidelines)
- [N/A] Document any change to the user interface, including configuration parameters (see node configuration)
- [N/A] Provide automatic testing (see the testing guide).
- [N/A] For new features and bug fixes, add an item in the appropriate changelog (
docs/protocols/alpha.rst
for the protocol and the environment,CHANGES.rst
at the root of the repository for everything else). -
Select suitable reviewers using the Reviewers
field below. -
Select as Assignee
the next person who should take action on that MR