docs: improve Execute code block (#534) (#555)

1. `InitialHeight` is specified twice 2. The contents of `Execute` refer to the parameter `state` not `s` (cherry picked from commit 9c5158a2f580a830d91dd3745f942916b673cfc6) Co-authored-by: Rootul P <[email protected]>
skip-mev · Mar 20, 2023 · f689ed5 · f689ed5
1 parent d722192
commit f689ed5
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/spec/core/data_structures.md b/spec/core/data_structures.md
@@ -71,7 +71,7 @@ and `ABCIApp` is an ABCI application that can return results and changes to the
 set (TODO). Execute is defined as:
 
 ```go
-func Execute(s State, app ABCIApp, block Block) State {
+func Execute(state State, app ABCIApp, block Block) State {
  // Fuction ApplyBlock executes block of transactions against the app and returns the new root hash of the app state,
  // modifications to the validator set and the changes of the consensus parameters.
  AppHash, ValidatorChanges, ConsensusParamChanges := app.ApplyBlock(block)
@@ -82,7 +82,6 @@ func Execute(s State, app ABCIApp, block Block) State {
   InitialHeight:   state.InitialHeight,
   LastResults:     abciResponses.DeliverTxResults,
   AppHash:         AppHash,
-  InitialHeight:   state.InitialHeight,
   LastValidators:  state.Validators,
   Validators:      state.NextValidators,
   NextValidators:  UpdateValidators(state.NextValidators, ValidatorChanges),