Skip to content

Commit 95f9444

Browse files
authored
Merge master (0cdf88e) into libevm (#823)
This is a `git merge -s ours master` with fixes in the merge commit.
2 parents 9c6f9cc + 7b07f71 commit 95f9444

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

62 files changed

+1801
-1321
lines changed

consensus/dummy/consensus.go

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -22,15 +22,15 @@ import (
2222
"github.com/ava-labs/libevm/trie"
2323

2424
customheader "github.com/ava-labs/coreth/plugin/evm/header"
25+
"github.com/ava-labs/coreth/plugin/evm/upgrade/ap1"
26+
"github.com/ava-labs/coreth/plugin/evm/upgrade/cortina"
2527
)
2628

2729
var (
2830
allowedFutureBlockTime = 10 * time.Second // Max time from current time allowed for blocks, before they're considered future blocks
2931

3032
errInvalidBlockTime = errors.New("timestamp less than parent's")
3133
errUnclesUnsupported = errors.New("uncles unsupported")
32-
errBlockGasCostNil = errors.New("block gas cost is nil")
33-
errBaseFeeNil = errors.New("base fee is nil")
3434
errExtDataGasUsedNil = errors.New("extDataGasUsed is nil")
3535
errExtDataGasUsedTooLarge = errors.New("extDataGasUsed is not uint64")
3636
)
@@ -124,12 +124,12 @@ func (eng *DummyEngine) verifyHeaderGasFields(config *params.ChainConfig, header
124124
return fmt.Errorf("invalid gasUsed: have %d, gasLimit %d", header.GasUsed, header.GasLimit)
125125
}
126126
if configExtra.IsCortina(header.Time) {
127-
if header.GasLimit != params.CortinaGasLimit {
128-
return fmt.Errorf("expected gas limit to be %d in Cortina, but found %d", params.CortinaGasLimit, header.GasLimit)
127+
if header.GasLimit != cortina.GasLimit {
128+
return fmt.Errorf("expected gas limit to be %d in Cortina, but found %d", cortina.GasLimit, header.GasLimit)
129129
}
130130
} else if configExtra.IsApricotPhase1(header.Time) {
131-
if header.GasLimit != params.ApricotPhase1GasLimit {
132-
return fmt.Errorf("expected gas limit to be %d in ApricotPhase1, but found %d", params.ApricotPhase1GasLimit, header.GasLimit)
131+
if header.GasLimit != ap1.GasLimit {
132+
return fmt.Errorf("expected gas limit to be %d in ApricotPhase1, but found %d", ap1.GasLimit, header.GasLimit)
133133
}
134134
} else {
135135
// Verify that the gas limit remains within allowed bounds
@@ -141,7 +141,7 @@ func (eng *DummyEngine) verifyHeaderGasFields(config *params.ChainConfig, header
141141
}
142142

143143
// Verify header.Extra matches the expected value.
144-
expectedExtraPrefix, err := CalcExtraPrefix(config, parent, header.Time)
144+
expectedExtraPrefix, err := customheader.ExtraPrefix(configExtra, parent, header.Time)
145145
if err != nil {
146146
return fmt.Errorf("failed to calculate extra prefix: %w", err)
147147
}
@@ -150,7 +150,7 @@ func (eng *DummyEngine) verifyHeaderGasFields(config *params.ChainConfig, header
150150
}
151151

152152
// Verify header.BaseFee matches the expected value.
153-
expectedBaseFee, err := CalcBaseFee(config, parent, header.Time)
153+
expectedBaseFee, err := customheader.BaseFee(configExtra, parent, header.Time)
154154
if err != nil {
155155
return fmt.Errorf("failed to calculate base fee: %w", err)
156156
}

consensus/dummy/consensus_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,8 @@ import (
99
"testing"
1010

1111
"github.com/ava-labs/coreth/core/types"
12-
"github.com/ava-labs/coreth/plugin/evm/ap4"
1312
"github.com/ava-labs/coreth/plugin/evm/header"
13+
"github.com/ava-labs/coreth/plugin/evm/upgrade/ap4"
1414
"github.com/ava-labs/libevm/common"
1515
)
1616

consensus/dummy/dynamic_fee_window.go

Lines changed: 0 additions & 88 deletions
This file was deleted.

consensus/dummy/dynamic_fee_window_test.go

Lines changed: 0 additions & 246 deletions
This file was deleted.

0 commit comments

Comments
 (0)