chore(core/types): Header libevm PostCopy hook

- Implement `PostCopy` header hook
- Remove `CopyHeader` and use imported one from libevm

See original PR ava-labs/coreth#759

Signed-off-by: Quentin McGaw <[email protected]>
Co-authored-by: Arran Schlosberg <[email protected]>

Author: qdm12

core/types/block.go

@@ -118,48 +118,6 @@ func NewBlock(
 	return b
 }
 
-// CopyHeader creates a deep copy of a block header.
-func CopyHeader(h *Header) *Header {
-	cpy := *h
-	hExtra := GetHeaderExtra(h)
-	cpyExtra := &HeaderExtra{}
-	SetHeaderExtra(&cpy, cpyExtra)
-
-	if cpy.Difficulty = new(big.Int); h.Difficulty != nil {
-		cpy.Difficulty.Set(h.Difficulty)
-	}
-	if cpy.Number = new(big.Int); h.Number != nil {
-		cpy.Number.Set(h.Number)
-	}
-	if h.BaseFee != nil {
-		cpy.BaseFee = new(big.Int).Set(h.BaseFee)
-	}
-	if hExtra.BlockGasCost != nil {
-		cpyExtra.BlockGasCost = new(big.Int).Set(hExtra.BlockGasCost)
-	}
-	if len(h.Extra) > 0 {
-		cpy.Extra = make([]byte, len(h.Extra))
-		copy(cpy.Extra, h.Extra)
-	}
-	if h.WithdrawalsHash != nil {
-		cpy.WithdrawalsHash = new(common.Hash)
-		*cpy.WithdrawalsHash = *h.WithdrawalsHash
-	}
-	if h.ExcessBlobGas != nil {
-		cpy.ExcessBlobGas = new(uint64)
-		*cpy.ExcessBlobGas = *h.ExcessBlobGas
-	}
-	if h.BlobGasUsed != nil {
-		cpy.BlobGasUsed = new(uint64)
-		*cpy.BlobGasUsed = *h.BlobGasUsed
-	}
-	if h.ParentBeaconRoot != nil {
-		cpy.ParentBeaconRoot = new(common.Hash)
-		*cpy.ParentBeaconRoot = *h.ParentBeaconRoot
-	}
-	return &cpy
-}
-
 // DecodeRLP decodes a block from RLP.
 func (b *Block) DecodeRLP(s *rlp.Stream) error {
 	var eb extblock

core/types/header_ext.go

@@ -82,7 +82,11 @@ func (h *HeaderExtra) DecodeJSON(eth *ethtypes.Header, input []byte) error {
 }
 
 func (h *HeaderExtra) PostCopy(dst *ethtypes.Header) {
-	panic("not implemented")
+	cp := &HeaderExtra{}
+	if h.BlockGasCost != nil {
+		cp.BlockGasCost = new(big.Int).Set(h.BlockGasCost)
+	}
+	SetHeaderExtra(dst, cp)
 }
 
 func (h *HeaderSerializable) updateFromEth(eth *ethtypes.Header) {

core/types/header_ext_test.go

@@ -177,4 +177,6 @@ func assertNonZero[T interface {
 	}
 }
 
+// Note [TestCopyHeader] tests the [HeaderExtra.PostCopy] method.
+
 func ptrTo[T any](x T) *T { return &x }

core/types/imports.go

@@ -52,6 +52,7 @@ const (
 var (
 	BloomLookup         = ethtypes.BloomLookup
 	BytesToBloom        = ethtypes.BytesToBloom
+	CopyHeader          = ethtypes.CopyHeader
 	CreateBloom         = ethtypes.CreateBloom
 	EncodeNonce         = ethtypes.EncodeNonce
 	FullAccount         = ethtypes.FullAccount