Use deque rather than vector for vote_cache::top results.

nanocurrency · May 17, 2024 · 5142047 · 5142047
1 parent 5e5cd37
commit 5142047
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/nano/node/vote_cache.cpp b/nano/node/vote_cache.cpp
@@ -231,11 +231,11 @@ void nano::vote_cache::clear ()
 	cache.clear ();
 }
 
-std::vector<nano::vote_cache::top_entry> nano::vote_cache::top (const nano::uint128_t & min_tally)
+std::deque<nano::vote_cache::top_entry> nano::vote_cache::top (const nano::uint128_t & min_tally)
 {
 	stats.inc (nano::stat::type::vote_cache, nano::stat::detail::top);
 
-	std::vector<top_entry> results;
+	std::deque<top_entry> results;
 	{
 		nano::lock_guard<nano::mutex> lock{ mutex };
 

diff --git a/nano/node/vote_cache.hpp b/nano/node/vote_cache.hpp
@@ -145,7 +145,7 @@ class vote_cache final
 	 * The blocks are sorted in descending order by final tally, then by tally
 	 * @param min_tally minimum tally threshold, entries below with their voting weight below this will be ignored
 	 */
-	std::vector<top_entry> top (nano::uint128_t const & min_tally);
+	std::deque<top_entry> top (nano::uint128_t const & min_tally);
 
 public: // Container info
 	std::unique_ptr<nano::container_info_component> collect_container_info (std::string const & name) const;