From 07db098ccf94ebfa80a9dedea85b4f55ad6f8d76 Mon Sep 17 00:00:00 2001
From: obscuren <geffobscura@gmail.com>
Date: Sat, 20 Jun 2015 15:56:34 +0200
Subject: core: renamed next to pending & fixed tests

---
 core/chain_manager.go      | 25 ++++++++++---------------
 core/chain_manager_test.go |  6 ++++--
 2 files changed, 14 insertions(+), 17 deletions(-)

(limited to 'core')

diff --git a/core/chain_manager.go b/core/chain_manager.go
index 3adaf3344..e8f4d9b5b 100644
--- a/core/chain_manager.go
+++ b/core/chain_manager.go
@@ -108,9 +108,9 @@ type ChainManager struct {
 	transState *state.StateDB
 	txState    *state.ManagedState
 
-	cache        *lru.Cache  // cache is the LRU caching
-	futureBlocks *BlockCache // future blocks are blocks added for later processing
-	nextBlocks   *BlockCache // next blocks is used during large inserts
+	cache         *lru.Cache  // cache is the LRU caching
+	futureBlocks  *BlockCache // future blocks are blocks added for later processing
+	pendingBlocks *BlockCache // pending blocks contain blocks not yet written to the db
 
 	quit chan struct{}
 	// procInterrupt must be atomically called
@@ -389,8 +389,8 @@ func (bc *ChainManager) HasBlock(hash common.Hash) bool {
 		return true
 	}
 
-	if bc.nextBlocks != nil {
-		if block := bc.nextBlocks.Get(hash); block != nil {
+	if bc.pendingBlocks != nil {
+		if block := bc.pendingBlocks.Get(hash); block != nil {
 			return true
 		}
 	}
@@ -425,8 +425,8 @@ func (self *ChainManager) GetBlock(hash common.Hash) *types.Block {
 		return block.(*types.Block)
 	}
 
-	if self.nextBlocks != nil {
-		if block := self.nextBlocks.Get(hash); block != nil {
+	if self.pendingBlocks != nil {
+		if block := self.pendingBlocks.Get(hash); block != nil {
 			return block
 		}
 	}
@@ -521,13 +521,13 @@ func (self *ChainManager) procFutureBlocks() {
 }
 
 func (self *ChainManager) enqueueForWrite(block *types.Block) {
-	self.nextBlocks.Push(block)
+	self.pendingBlocks.Push(block)
 }
 
 func (self *ChainManager) flushQueuedBlocks() {
 	db, batchWrite := self.blockDb.(*ethdb.LDBDatabase)
 	batch := new(leveldb.Batch)
-	self.nextBlocks.Each(func(i int, block *types.Block) {
+	self.pendingBlocks.Each(func(i int, block *types.Block) {
 		enc, _ := rlp.EncodeToBytes((*types.StorageBlock)(block))
 		key := append(blockHashPre, block.Hash().Bytes()...)
 		if batchWrite {
@@ -539,10 +539,6 @@ func (self *ChainManager) flushQueuedBlocks() {
 	if batchWrite {
 		db.LDB().Write(batch, nil)
 	}
-
-	// reset the next blocks cache
-	self.nextBlocks = nil
-
 }
 
 // InsertChain will attempt to insert the given chain in to the canonical chain or, otherwise, create a fork. It an error is returned
@@ -554,7 +550,7 @@ func (self *ChainManager) InsertChain(chain types.Blocks) (int, error) {
 	self.chainmu.Lock()
 	defer self.chainmu.Unlock()
 
-	self.nextBlocks = NewBlockCache(len(chain))
+	self.pendingBlocks = NewBlockCache(len(chain))
 
 	// A queued approach to delivering events. This is generally
 	// faster than direct delivery and requires much less mutex
@@ -687,7 +683,6 @@ func (self *ChainManager) InsertChain(chain types.Blocks) (int, error) {
 			queue[i] = ChainSideEvent{block, logs}
 			queueEvent.sideCount++
 		}
-		// Write block to database. Eventually we'll have to improve on this and throw away blocks that are
 		// not in the canonical chain.
 		self.enqueueForWrite(block)
 		// Delete from future blocks
diff --git a/core/chain_manager_test.go b/core/chain_manager_test.go
index c37cd1a5e..70130c02c 100644
--- a/core/chain_manager_test.go
+++ b/core/chain_manager_test.go
@@ -18,6 +18,7 @@ import (
 	"github.com/ethereum/go-ethereum/event"
 	"github.com/ethereum/go-ethereum/pow"
 	"github.com/ethereum/go-ethereum/rlp"
+	"github.com/hashicorp/golang-lru"
 )
 
 func init() {
@@ -109,7 +110,8 @@ func testChain(chainB types.Blocks, bman *BlockProcessor) (*big.Int, error) {
 
 		bman.bc.mu.Lock()
 		{
-			bman.bc.write(block)
+			bman.bc.enqueueForWrite(block)
+			//bman.bc.write(block)
 		}
 		bman.bc.mu.Unlock()
 	}
@@ -391,7 +393,7 @@ func makeChainWithDiff(genesis *types.Block, d []int, seed byte) []*types.Block
 func chm(genesis *types.Block, db common.Database) *ChainManager {
 	var eventMux event.TypeMux
 	bc := &ChainManager{blockDb: db, stateDb: db, genesisBlock: genesis, eventMux: &eventMux, pow: FakePow{}}
-	bc.cache = NewBlockCache(100)
+	bc.cache, _ = lru.New(100)
 	bc.futureBlocks = NewBlockCache(100)
 	bc.processor = bproc{}
 	bc.ResetWithGenesisBlock(genesis)
-- 
cgit v1.2.3