From d8cf87949991c0ee5ef504af4bb398cce524deaf Mon Sep 17 00:00:00 2001 From: Roman Khimov Date: Fri, 18 Feb 2022 11:41:27 +0300 Subject: [PATCH] dao: deduplicate DeleteBlock, no functional changes --- pkg/core/dao/dao.go | 15 +++++---------- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/pkg/core/dao/dao.go b/pkg/core/dao/dao.go index b6f6d3829..cd4221e1b 100644 --- a/pkg/core/dao/dao.go +++ b/pkg/core/dao/dao.go @@ -375,7 +375,10 @@ func (dao *Simple) makeStorageItemKey(id int32, key []byte) []byte { // GetBlock returns Block by the given hash if it exists in the store. func (dao *Simple) GetBlock(hash util.Uint256) (*block.Block, error) { - key := dao.makeExecutableKey(hash) + return dao.getBlock(dao.makeExecutableKey(hash)) +} + +func (dao *Simple) getBlock(key []byte) (*block.Block, error) { b, err := dao.Store.Get(key) if err != nil { return nil, err @@ -659,16 +662,8 @@ func (dao *Simple) StoreAsBlock(block *block.Block, aer1 *state.AppExecResult, a // using private MemCached instance here. func (dao *Simple) DeleteBlock(h util.Uint256) error { key := dao.makeExecutableKey(h) - bs, err := dao.Store.Get(key) - if err != nil { - return err - } - r := io.NewBinReaderFromBuf(bs) - if r.ReadB() != storage.ExecBlock { - return errors.New("internal DB inconsistency") - } - b, err := block.NewTrimmedFromReader(dao.Version.StateRootInHeader, r) + b, err := dao.getBlock(key) if err != nil { return err }