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

x/evm/types: use ethtypes.Transaction.To for cheaper retrieval #835

Merged
merged 1 commit into from
Dec 14, 2021

Conversation

odeke-em
Copy link
Contributor

Following suit with PR #828, this change cuts down the expenses
from using .To doubly; yet using the Go in-condition variable idiom.

Updates #826

Following suit with PR #828, this change cuts down the expenses
from using .To doubly; yet using the Go in-condition variable idiom.

Updates #826
@codecov
Copy link

codecov bot commented Dec 14, 2021

Codecov Report

Merging #835 (d21c656) into main (a2f246c) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #835   +/-   ##
=======================================
  Coverage   57.31%   57.31%           
=======================================
  Files          72       72           
  Lines        5984     5984           
=======================================
  Hits         3430     3430           
  Misses       2357     2357           
  Partials      197      197           
Impacted Files Coverage Δ
x/evm/types/access_list_tx.go 70.94% <100.00%> (ø)
x/evm/types/dynamic_fee_tx.go 78.61% <100.00%> (ø)
x/evm/types/legacy_tx.go 95.20% <100.00%> (ø)

@odeke-em odeke-em merged commit 423944b into main Dec 14, 2021
@odeke-em odeke-em deleted the x-evm-types-use-cheap-To branch December 14, 2021 01:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants