rottenwheel/monero
1
0
Fork 0
mirror of https://codeberg.org/anoncontributorxmr/monero.git synced 2024-11-23 10:37:37 -07:00
Code Issues Actions Packages Projects Releases Wiki Activity

Remove copies from foreach loops (thanks to Clang)

Browse Source
This commit is contained in:
Lee Clagett 2021-01-28 23:42:40 -05:00
parent 1572df9e26
commit bd129849f0
8 changed files with 17 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/blockchain_db/blockchain_db.cpp
View File

@ -287,7 +287,7 @@ uint64_t BlockchainDB::add_block( const std::pair<block, blobdata>& blck
num_rct_outs += blk.miner_tx.vout.size(); num_rct_outs += blk.miner_tx.vout.size();
int tx_i = 0; int tx_i = 0;
crypto::hash tx_hash = crypto::null_hash; crypto::hash tx_hash = crypto::null_hash;
for (const std::pair<transaction, blobdata_ref>& tx : txs) for (const std::pair<transaction, blobdata>& tx : txs)
{ {
tx_hash = blk.tx_hashes[tx_i]; tx_hash = blk.tx_hashes[tx_i];
add_transaction(blk_hash, tx, &tx_hash); add_transaction(blk_hash, tx, &tx_hash);

4
src/blockchain_utilities/blockchain_ancestry.cpp
View File

@ -149,7 +149,7 @@ struct ancestry_state_t
{ {
std::unordered_map<crypto::hash, cryptonote::transaction> old_tx_cache; std::unordered_map<crypto::hash, cryptonote::transaction> old_tx_cache;
a & old_tx_cache; a & old_tx_cache;
for (const auto i: old_tx_cache) for (const auto& i: old_tx_cache)
tx_cache.insert(std::make_pair(i.first, ::tx_data_t(i.second))); tx_cache.insert(std::make_pair(i.first, ::tx_data_t(i.second)));
} }
else else
@ -161,7 +161,7 @@ struct ancestry_state_t
std::unordered_map<uint64_t, cryptonote::block> old_block_cache; std::unordered_map<uint64_t, cryptonote::block> old_block_cache;
a & old_block_cache; a & old_block_cache;
block_cache.resize(old_block_cache.size()); block_cache.resize(old_block_cache.size());
for (const auto i: old_block_cache) for (const auto& i: old_block_cache)
block_cache[i.first] = i.second; block_cache[i.first] = i.second;
} }
else else

2
src/cryptonote_core/blockchain.cpp
View File

@ -973,7 +973,7 @@ start:
std::pair<bool, uint64_t> Blockchain::check_difficulty_checkpoints() const std::pair<bool, uint64_t> Blockchain::check_difficulty_checkpoints() const
{ {
uint64_t res = 0; uint64_t res = 0;
for (const std::pair<uint64_t, difficulty_type>& i : m_checkpoints.get_difficulty_points()) for (const std::pair<const uint64_t, difficulty_type>& i : m_checkpoints.get_difficulty_points())
{ {
if (i.first >= m_db->height()) if (i.first >= m_db->height())
break; break;

2
src/cryptonote_core/cryptonote_core.cpp
View File

@ -630,7 +630,7 @@ namespace cryptonote
void operator()(std::uint64_t, epee::span<const block> blocks) const void operator()(std::uint64_t, epee::span<const block> blocks) const
{ {
for (const block bl : blocks) for (const block& bl : blocks)
cmdline.notify("%s", epee::string_tools::pod_to_hex(get_block_hash(bl)).c_str(), NULL); cmdline.notify("%s", epee::string_tools::pod_to_hex(get_block_hash(bl)).c_str(), NULL);
} }
}; };

2
src/simplewallet/simplewallet.cpp
View File

@ -9466,7 +9466,7 @@ void simple_wallet::print_accounts()
{ {
const std::pair<std::map<std::string, std::string>, std::vector<std::string>>& account_tags = m_wallet->get_account_tags(); const std::pair<std::map<std::string, std::string>, std::vector<std::string>>& account_tags = m_wallet->get_account_tags();
size_t num_untagged_accounts = m_wallet->get_num_subaddress_accounts(); size_t num_untagged_accounts = m_wallet->get_num_subaddress_accounts();
for (const std::pair<std::string, std::string>& p : account_tags.first) for (const std::pair<const std::string, std::string>& p : account_tags.first)
{ {
const std::string& tag = p.first; const std::string& tag = p.first;
print_accounts(tag); print_accounts(tag);

2
src/wallet/wallet_rpc_server.cpp
View File

@ -687,7 +687,7 @@ namespace tools
{ {
if (!m_wallet) return not_open(er); if (!m_wallet) return not_open(er);
const std::pair<std::map<std::string, std::string>, std::vector<std::string>> account_tags = m_wallet->get_account_tags(); const std::pair<std::map<std::string, std::string>, std::vector<std::string>> account_tags = m_wallet->get_account_tags();
for (const std::pair<std::string, std::string>& p : account_tags.first) for (const std::pair<const std::string, std::string>& p : account_tags.first)
{ {
res.account_tags.resize(res.account_tags.size() + 1); res.account_tags.resize(res.account_tags.size() + 1);
auto& info = res.account_tags.back(); auto& info = res.account_tags.back();

4
tests/core_tests/tx_pool.cpp
View File

@ -393,7 +393,7 @@ bool txpool_double_spend_base::check_changed(cryptonote::core& c, const size_t e
} }
} }
for (const std::pair<crypto::hash, uint64_t>& hash : m_all_hashes) for (const std::pair<const crypto::hash, uint64_t>& hash : m_all_hashes)
{ {
cryptonote::blobdata tx_blob{}; cryptonote::blobdata tx_blob{};
if (!c.get_pool_transaction(hash.first, tx_blob, cryptonote::relay_category::all)) if (!c.get_pool_transaction(hash.first, tx_blob, cryptonote::relay_category::all))
@ -411,7 +411,7 @@ bool txpool_double_spend_base::check_changed(cryptonote::core& c, const size_t e
for (const crypto::hash& hash : m_no_relay_hashes) for (const crypto::hash& hash : m_no_relay_hashes)
difference.erase(hash); difference.erase(hash);
for (const std::pair<crypto::hash, uint64_t>& hash : difference) for (const std::pair<const crypto::hash, uint64_t>& hash : difference)
{ {
if (c.pool_has_tx(hash.first)) if (c.pool_has_tx(hash.first))
{ {

16
tests/unit_tests/net.cpp
View File

@ -1351,7 +1351,7 @@ TEST(dandelionpp_map, dropped_connection)
} }
EXPECT_EQ(3u, used.size()); EXPECT_EQ(3u, used.size());
for (const std::pair<boost::uuids::uuid, std::size_t>& entry : used) for (const std::pair<const boost::uuids::uuid, std::size_t>& entry : used)
EXPECT_EQ(3u, entry.second); EXPECT_EQ(3u, entry.second);
for (const boost::uuids::uuid& connection : in_connections) for (const boost::uuids::uuid& connection : in_connections)
@ -1409,7 +1409,7 @@ TEST(dandelionpp_map, dropped_connection)
} }
EXPECT_EQ(3u, used.size()); EXPECT_EQ(3u, used.size());
for (const std::pair<boost::uuids::uuid, std::size_t>& entry : used) for (const std::pair<const boost::uuids::uuid, std::size_t>& entry : used)
EXPECT_EQ(3u, entry.second); EXPECT_EQ(3u, entry.second);
} }
{ {
@ -1472,7 +1472,7 @@ TEST(dandelionpp_map, dropped_connection_remapped)
} }
EXPECT_EQ(3u, used.size()); EXPECT_EQ(3u, used.size());
for (const std::pair<boost::uuids::uuid, std::size_t>& entry : used) for (const std::pair<const boost::uuids::uuid, std::size_t>& entry : used)
EXPECT_EQ(3u, entry.second); EXPECT_EQ(3u, entry.second);
for (const boost::uuids::uuid& connection : in_connections) for (const boost::uuids::uuid& connection : in_connections)
@ -1511,7 +1511,7 @@ TEST(dandelionpp_map, dropped_connection_remapped)
} }
EXPECT_EQ(2u, used.size()); EXPECT_EQ(2u, used.size());
for (const std::pair<boost::uuids::uuid, std::size_t>& entry : used) for (const std::pair<const boost::uuids::uuid, std::size_t>& entry : used)
EXPECT_EQ(5u, entry.second); EXPECT_EQ(5u, entry.second);
} }
// select 3 of 3 connections but do not remap existing links // select 3 of 3 connections but do not remap existing links
@ -1532,7 +1532,7 @@ TEST(dandelionpp_map, dropped_connection_remapped)
} }
EXPECT_EQ(2u, used.size()); EXPECT_EQ(2u, used.size());
for (const std::pair<boost::uuids::uuid, std::size_t>& entry : used) for (const std::pair<const boost::uuids::uuid, std::size_t>& entry : used)
EXPECT_EQ(5u, entry.second); EXPECT_EQ(5u, entry.second);
} }
// map 8 new incoming connections across 3 outgoing links // map 8 new incoming connections across 3 outgoing links
@ -1555,7 +1555,7 @@ TEST(dandelionpp_map, dropped_connection_remapped)
} }
EXPECT_EQ(3u, used.size()); EXPECT_EQ(3u, used.size());
for (const std::pair<boost::uuids::uuid, std::size_t>& entry : used) for (const std::pair<const boost::uuids::uuid, std::size_t>& entry : used)
EXPECT_EQ(6u, entry.second); EXPECT_EQ(6u, entry.second);
} }
} }
@ -1609,7 +1609,7 @@ TEST(dandelionpp_map, dropped_all_connections)
} }
EXPECT_EQ(3u, used.size()); EXPECT_EQ(3u, used.size());
for (const std::pair<boost::uuids::uuid, std::size_t>& entry : used) for (const std::pair<const boost::uuids::uuid, std::size_t>& entry : used)
EXPECT_EQ(3u, entry.second); EXPECT_EQ(3u, entry.second);
for (const boost::uuids::uuid& connection : in_connections) for (const boost::uuids::uuid& connection : in_connections)
@ -1641,7 +1641,7 @@ TEST(dandelionpp_map, dropped_all_connections)
} }
EXPECT_EQ(3u, used.size()); EXPECT_EQ(3u, used.size());
for (const std::pair<boost::uuids::uuid, std::size_t>& entry : used) for (const std::pair<const boost::uuids::uuid, std::size_t>& entry : used)
EXPECT_EQ(3u, entry.second); EXPECT_EQ(3u, entry.second);
} }
} }