From bb7a5b4d6d8a36d1287f04adbf063e29be2bc73a Mon Sep 17 00:00:00 2001 From: SChernykh Date: Tue, 2 Nov 2021 16:48:46 +0100 Subject: [PATCH] Cleanup --- src/pool_block.cpp | 12 +++++------- src/pool_block.h | 1 - src/pool_block_parser.inl | 1 - tests/src/pool_block_tests.cpp | 1 - 4 files changed, 5 insertions(+), 10 deletions(-) diff --git a/src/pool_block.cpp b/src/pool_block.cpp index e9e884fb..5003559d 100644 --- a/src/pool_block.cpp +++ b/src/pool_block.cpp @@ -60,7 +60,6 @@ PoolBlock::PoolBlock() m_mainChainData.reserve(48 * 1024); m_outputs.reserve(2048); m_transactions.reserve(256); - m_tmpInts.reserve(m_transactions.capacity() * HASH_SIZE); m_sideChainData.reserve(512); m_uncles.reserve(8); m_tmpTxExtra.reserve(80); @@ -110,7 +109,6 @@ PoolBlock& PoolBlock::operator=(const PoolBlock& b) m_cumulativeDifficulty = b.m_cumulativeDifficulty; m_sidechainId = b.m_sidechainId; m_tmpTxExtra = b.m_tmpTxExtra; - m_tmpInts = b.m_tmpInts; m_depth = b.m_depth; m_verified = b.m_verified; m_invalid = b.m_invalid; @@ -276,21 +274,21 @@ bool PoolBlock::get_pow_hash(RandomX_Hasher* hasher, const hash& seed_hash, hash cnt >>= 1; - m_tmpInts.resize(cnt * HASH_SIZE); - memcpy(m_tmpInts.data(), h, (cnt * 2 - count) * HASH_SIZE); + std::vector tmp_ints(cnt * HASH_SIZE); + memcpy(tmp_ints.data(), h, (cnt * 2 - count) * HASH_SIZE); for (i = cnt * 2 - count, j = cnt * 2 - count; j < cnt; i += 2, ++j) { - keccak(h + i * HASH_SIZE, HASH_SIZE * 2, m_tmpInts.data() + j * HASH_SIZE, HASH_SIZE); + keccak(h + i * HASH_SIZE, HASH_SIZE * 2, tmp_ints.data() + j * HASH_SIZE, HASH_SIZE); } while (cnt > 2) { cnt >>= 1; for (i = 0, j = 0; j < cnt; i += 2, ++j) { - keccak(m_tmpInts.data() + i * HASH_SIZE, HASH_SIZE * 2, m_tmpInts.data() + j * HASH_SIZE, HASH_SIZE); + keccak(tmp_ints.data() + i * HASH_SIZE, HASH_SIZE * 2, tmp_ints.data() + j * HASH_SIZE, HASH_SIZE); } } - keccak(m_tmpInts.data(), HASH_SIZE * 2, blob + blob_size, HASH_SIZE); + keccak(tmp_ints.data(), HASH_SIZE * 2, blob + blob_size, HASH_SIZE); } } blob_size += HASH_SIZE; diff --git a/src/pool_block.h b/src/pool_block.h index 8a915331..1430be85 100644 --- a/src/pool_block.h +++ b/src/pool_block.h @@ -121,7 +121,6 @@ struct PoolBlock // Just temporary stuff, not a part of the block std::vector m_tmpTxExtra; - std::vector m_tmpInts; uint64_t m_depth; diff --git a/src/pool_block_parser.inl b/src/pool_block_parser.inl index 5ec637ce..a2b3d6b3 100644 --- a/src/pool_block_parser.inl +++ b/src/pool_block_parser.inl @@ -341,7 +341,6 @@ int PoolBlock::deserialize(const uint8_t* data, size_t size, SideChain& sidechai // Defaults for off-chain variables m_tmpTxExtra.clear(); - m_tmpInts.clear(); m_depth = 0; diff --git a/tests/src/pool_block_tests.cpp b/tests/src/pool_block_tests.cpp index 2fba68c5..55dde77e 100644 --- a/tests/src/pool_block_tests.cpp +++ b/tests/src/pool_block_tests.cpp @@ -73,7 +73,6 @@ TEST(pool_block, deserialize) ASSERT_EQ(b.m_cumulativeDifficulty.lo, 12544665764606ull); ASSERT_EQ(b.m_cumulativeDifficulty.hi, 0); ASSERT_EQ(b.m_tmpTxExtra.size(), 0); - ASSERT_EQ(b.m_tmpInts.size(), 0); ASSERT_EQ(b.m_depth, 0); ASSERT_EQ(b.m_verified, false); ASSERT_EQ(b.m_invalid, false);