Remove by tally index

pwojcikdev · Mar 23, 2024 · aa64459 · aa64459
1 parent 97d4dac
commit aa64459
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 8 deletions.
diff --git a/nano/node/vote_cache.cpp b/nano/node/vote_cache.cpp
@@ -244,13 +244,12 @@ std::vector<nano::vote_cache::top_entry> nano::vote_cache::top (const nano::uint
 			cleanup ();
 		}
 
-		for (auto & entry : cache.get<tag_tally> ())
+		for (auto & entry : cache)
 		{
-			if (entry.tally () < min_tally)
+			if (entry.tally () >= min_tally)
 			{
-				break;
+				results.push_back ({ entry.hash (), entry.tally (), entry.final_tally () });
 			}
-			results.push_back ({ entry.hash (), entry.tally (), entry.final_tally () });
 		}
 	}
 

diff --git a/nano/node/vote_cache.hpp b/nano/node/vote_cache.hpp
@@ -164,17 +164,14 @@ class vote_cache final
 	// clang-format off
 	class tag_sequenced {};
 	class tag_hash {};
-	class tag_tally {};
 	// clang-format on
 
 	// clang-format off
 	using ordered_cache = boost::multi_index_container<entry,
 	mi::indexed_by<
 		mi::hashed_unique<mi::tag<tag_hash>,
 			mi::const_mem_fun<entry, nano::block_hash, &entry::hash>>,
-		mi::sequenced<mi::tag<tag_sequenced>>,
-		mi::ordered_non_unique<mi::tag<tag_tally>,
-			mi::const_mem_fun<entry, nano::uint128_t, &entry::tally>, std::greater<>> // DESC
+		mi::sequenced<mi::tag<tag_sequenced>>
 	>>;
 	// clang-format on
 	ordered_cache cache;