fix: replace HashSet with BTreeSet

Author: rflechtner

pallets/pallet-bonded-coins/src/lib.rs

@@ -70,7 +70,6 @@ pub mod pallet {
 		Hashable, Parameter,
 	};
 	use frame_system::pallet_prelude::*;
-	use parity_scale_codec::alloc::collections::HashSet;
 	use sp_arithmetic::ArithmeticError;
 	use sp_core::U256;
 	use sp_runtime::{
@@ -80,6 +79,7 @@ pub mod pallet {
 		BoundedVec, DispatchError, TokenError,
 	};
 	use sp_std::{
+		collections::btree_set::BTreeSet,
 		iter::Iterator,
 		ops::{AddAssign, BitOrAssign, ShlAssign},
 		prelude::*,
@@ -408,8 +408,8 @@ pub mod pallet {
 			// currency to it. This should also verify that the currency actually exists.
 			T::Collaterals::touch(collateral_id.clone(), pool_account, &who)?;
 
-			let mut names_seen = HashSet::<StringInputOf<T>>::with_capacity(currencies.len());
-			let mut symbols_seen = HashSet::<StringInputOf<T>>::with_capacity(currencies.len());
+			let mut names_seen = BTreeSet::<StringInputOf<T>>::new();
+			let mut symbols_seen = BTreeSet::<StringInputOf<T>>::new();
 
 			currencies.into_iter().zip(currency_ids.iter()).try_for_each(
 				|(token_metadata, asset_id)| -> DispatchResult {