remove BlockWithExtData in favor of WithBlockExtras taking a recalc argument

Author: qdm12

core/rawdb/accessors_chain.go

@@ -524,7 +524,7 @@ func ReadBlock(db ethdb.Reader, hash common.Hash, number uint64) *types.Block {
 	}
 	block := types.NewBlockWithHeader(header).WithBody(body.Transactions, body.Uncles)
 	bodyExtra := types.GetBodyExtra(body)
-	block = types.BlockWithExtData(block, bodyExtra.Version, bodyExtra.ExtData)
+	block = types.WithBlockExtras(block, bodyExtra.Version, bodyExtra.ExtData, false)
 	return block
 }
 

core/types/block_ext.go

@@ -92,33 +92,6 @@ func (b *blockSerializable) updateToExtras(extras *BlockExtra) {
 	extras.extdata = b.ExtData
 }
 
-func BlockWithExtData(b *Block, version uint32, extdata *[]byte) *Block {
-	BlockExtras(b).version = version
-	setExtDataHelper(b, extdata, false)
-	return b
-}
-
-func setExtDataHelper(b *Block, data *[]byte, recalc bool) {
-	if data == nil {
-		setExtData(b, nil, recalc)
-		return
-	}
-	setExtData(b, *data, recalc)
-}
-
-func setExtData(b *Block, data []byte, recalc bool) {
-	_data := make([]byte, len(data))
-	extras := BlockExtras(b)
-	extras.extdata = &_data
-	copy(*extras.extdata, data)
-	if recalc {
-		header := b.Header()
-		headerExtra := HeaderExtras(header)
-		headerExtra.ExtDataHash = CalcExtDataHash(_data)
-		b.SetHeader(header)
-	}
-}
-
 func BlockExtData(b *Block) []byte {
 	extras := BlockExtras(b)
 	if extras.extdata == nil {
@@ -155,7 +128,7 @@ func NewBlockWithExtData(
 	header *Header, txs []*Transaction, uncles []*Header, receipts []*Receipt,
 	hasher TrieHasher, extdata []byte, recalc bool,
 ) *Block {
-	b := ethtypes.NewBlock(header, txs, uncles, receipts, hasher)
+	b := NewBlock(header, txs, uncles, receipts, hasher)
 	const version = 0
 	return WithBlockExtras(b, version, &extdata, recalc)
 }

core/types/libevm.go

@@ -45,7 +45,23 @@ func BlockExtras(b *Block) *BlockExtra {
 }
 
 func WithBlockExtras(b *Block, version uint32, extdata *[]byte, recalc bool) *Block {
-	BlockExtras(b).version = version
-	setExtDataHelper(b, extdata, recalc)
+	extras := BlockExtras(b)
+
+	extras.version = version
+
+	cpy := make([]byte, 0)
+	if extdata != nil {
+		cpy = make([]byte, len(*extdata))
+		copy(cpy, *extdata)
+	}
+
+	extras.extdata = &cpy
+	if recalc {
+		header := b.Header()
+		headerExtra := HeaderExtras(header)
+		headerExtra.ExtDataHash = CalcExtDataHash(cpy)
+		b.SetHeader(header)
+	}
+
 	return b
 }