Skip to content
This repository has been archived by the owner on Apr 4, 2024. It is now read-only.

fix: traceTransaction returns zero gas used (backport #1179) #1200

Closed
wants to merge 3 commits into from
Closed

fix: traceTransaction returns zero gas used (backport #1179) #1200

wants to merge 3 commits into from

Commits on Jul 25, 2022

  1. fix: traceTransaction returns zero gas used (#1179) 

    * Problem: traceTransaction returns zero gas used

Solution:
- call CaptureTxStart and CaptureTxEnd

* fix trace unit tests

* add overflow check

* add comment

* check gas used is positive

* Update x/evm/keeper/state_transition.go

Co-authored-by: Federico Kunze Küllmer <[email protected]>
(cherry picked from commit b74b37f)

# Conflicts:
#	x/evm/keeper/grpc_query_test.go
    @yihuang @mergify
    yihuang authored and mergify[bot] committed Jul 25, 2022
    Configuration menu
    Copy the full SHA
    095bfb3 View commit details
    Browse the repository at this point in the history

  2. changelog

    @fedekunze
    fedekunze committed Jul 25, 2022
    Configuration menu
    Copy the full SHA
    5b7b424 View commit details
    Browse the repository at this point in the history

  3. conflicts

    @fedekunze
    fedekunze committed Jul 25, 2022
    Configuration menu
    Copy the full SHA
    8af1224 View commit details
    Browse the repository at this point in the history