Problem: ForEachStorage sematic not compatible with go-ethereum

[yihuang]

Solution:

• reversed the semantic of return value of the callback function.

ref: https://github.com/ethereum/go-ethereum/blob/master/core/state/statedb.go#L637

Description

---

For contributor use:

• Targeted PR against correct branch (see CONTRIBUTING.md)
• Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
• Code follows the module structure standards.
• Wrote unit and integration tests
• Updated relevant documentation (docs/) or specification (x/<module>/spec/)
• Added relevant godoc comments.
• Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
• Re-reviewed Files changed in the Github PR explorer

---

For admin use:

• Added appropriate labels to PR (ex. WIP, R4R, docs, etc)
• Reviewers assigned
• Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

[fedekunze]

Good catch, can you add a bugfix changelog entry?

[yihuang]

Good catch, can you add a bugfix changelog entry?

done

[codecov]

Codecov Report

Merging #798 (69fd5c0) into main (5a49153) will not change coverage.
The diff coverage is 33.33%.

@@           Coverage Diff           @@
##             main     #798   +/-   ##
=======================================
  Coverage   57.23%   57.23%           
=======================================
  Files          73       73           
  Lines        6021     6021           
=======================================
  Hits         3446     3446           
  Misses       2375     2375           
  Partials      200      200           

Impacted Files	Coverage Δ
x/evm/keeper/keeper.go	75.86% <0.00%> (ø)
x/evm/keeper/statedb.go	73.26% <100.00%> (ø)