aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorThomas Winget <tewinget@gmail.com>2015-01-14 17:41:46 -0500
committerThomas Winget <tewinget@gmail.com>2015-01-14 17:41:46 -0500
commit40e0b70075608cae9810de1667cdfe0b879049e1 (patch)
tree901b3b7d434cfc01b2954ba84a5fab9bed75fe46
parentMerge pull request #21 from warptangent/fix_block_reorg (diff)
parentRemove code previously made unused and marked unused (diff)
downloadmonero-40e0b70075608cae9810de1667cdfe0b879049e1.tar.xz
Merge #22
Remove code previously made unused and marked unused
-rw-r--r--src/cryptonote_core/blockchain.cpp18
-rw-r--r--src/cryptonote_core/blockchain_db.cpp8
2 files changed, 0 insertions, 26 deletions
diff --git a/src/cryptonote_core/blockchain.cpp b/src/cryptonote_core/blockchain.cpp
index 4fa868abe..88ae9d136 100644
--- a/src/cryptonote_core/blockchain.cpp
+++ b/src/cryptonote_core/blockchain.cpp
@@ -2028,24 +2028,6 @@ bool Blockchain::handle_block_to_main_chain(const block& bl, const crypto::hash&
{
LOG_PRINT_L3("Blockchain::" << __func__);
- // NOTE: Omitting check below with have_block() It causes an error after a
- // blockchain reorganize begins with error: "Attempting to add block to main
- // chain, but it's already either there or in an alternate"
- //
- // A block in the alternative chain, desired to become the main chain, never
- // makes it due to have_block finding it in he alternative chain.
- //
- // Original implementation didn't use it here, and it doesn't appear
- // necessary to be called from here in this implementation either.
-
- // if we already have the block, return false
- // if (have_block(id))
- // {
- // LOG_PRINT_L0("Attempting to add block to main chain, but it's already either there or in an alternate chain. hash: " << id);
- // bvc.m_verifivation_failed = true;
- // return false;
- // }
-
TIME_MEASURE_START(block_processing_time);
CRITICAL_REGION_LOCAL(m_blockchain_lock);
if(bl.prev_id != get_tail_id())
diff --git a/src/cryptonote_core/blockchain_db.cpp b/src/cryptonote_core/blockchain_db.cpp
index 5e781af79..0ee10bd4d 100644
--- a/src/cryptonote_core/blockchain_db.cpp
+++ b/src/cryptonote_core/blockchain_db.cpp
@@ -107,14 +107,6 @@ void BlockchainDB::remove_transaction(const crypto::hash& tx_hash)
{
transaction tx = get_tx(tx_hash);
- // TODO: This loop calling remove_output() should be removed. It doesn't do
- // anything (function is empty), and the outputs were already intended to be
- // removed later as part of remove_transaction_data().
- for (const tx_out& tx_output : tx.vout)
- {
- remove_output(tx_output);
- }
-
for (const txin_v& tx_input : tx.vin)
{
if (tx_input.type() == typeid(txin_to_key))