Skip to content

Commit

Permalink
Merge pull request #16945 from karalabe/triedb-spurious-warning
Browse files Browse the repository at this point in the history
trie: don't report the root flushlist as an alloc
  • Loading branch information
karalabe authored Jun 11, 2018
2 parents b487bdf + a3267ed commit 43b940e
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions trie/database.go
Original file line number Diff line number Diff line change
Expand Up @@ -297,7 +297,7 @@ func (db *Database) Cap(limit common.StorageSize) error {
// db.nodesSize only contains the useful data in the cache, but when reporting
// the total memory consumption, the maintenance metadata is also needed to be
// counted. For every useful node, we track 2 extra hashes as the flushlist.
size := db.nodesSize + common.StorageSize(len(db.nodes)*2*common.HashLength)
size := db.nodesSize + common.StorageSize((len(db.nodes)-1)*2*common.HashLength)

// If the preimage cache got large enough, push to disk. If it's still small
// leave for later to deduplicate writes.
Expand Down Expand Up @@ -512,6 +512,6 @@ func (db *Database) Size() (common.StorageSize, common.StorageSize) {
// db.nodesSize only contains the useful data in the cache, but when reporting
// the total memory consumption, the maintenance metadata is also needed to be
// counted. For every useful node, we track 2 extra hashes as the flushlist.
var flushlistSize = common.StorageSize(len(db.nodes) * 2 * common.HashLength)
var flushlistSize = common.StorageSize((len(db.nodes) - 1) * 2 * common.HashLength)
return db.nodesSize + flushlistSize, db.preimagesSize
}

0 comments on commit 43b940e

Please sign in to comment.