Revert "Uses custom bound for AsPrimitive"

This reverts commit 838e528.

bridges/bin/runtime-common/src/refund_relayer_extension.rs

@@ -47,10 +47,7 @@ use pallet_transaction_payment::{Config as TransactionPaymentConfig, OnChargeTra
 use pallet_utility::{Call as UtilityCall, Config as UtilityConfig, Pallet as UtilityPallet};
 use scale_info::TypeInfo;
 use sp_runtime::{
-	traits::{
-		Block as BlockT, DispatchInfoOf, Get, Header as HeaderT, PostDispatchInfoOf,
-		SignedExtension, Zero,
-	},
+	traits::{Block as BlockT, Header as HeaderT, DispatchInfoOf, Get, PostDispatchInfoOf, SignedExtension, Zero},
 	transaction_validity::{
 		TransactionPriority, TransactionValidity, TransactionValidityError, ValidTransactionBuilder,
 	},

bridges/modules/parachains/src/lib.rs

@@ -139,15 +139,18 @@ pub mod pallet {
 	pub trait BoundedBridgeGrandpaConfig<I: 'static>:
 		pallet_bridge_grandpa::Config<I, BridgedChain = Self::BridgedRelayChain>
 	{
-		type BridgedRelayChain: Chain<Hash = RelayBlockHash, Hasher = RelayBlockHasher>;
+		type BridgedRelayChain: Chain<
+			Hash = RelayBlockHash,
+			Hasher = RelayBlockHasher,
+		>;
 	}
 
 	impl<T, I: 'static> BoundedBridgeGrandpaConfig<I> for T
 	where
 		T: pallet_bridge_grandpa::Config<I>,
-		T::BridgedChain: Chain<Hash = RelayBlockHash, Hasher = RelayBlockHasher>,
-		<<T::BridgedChain as Chain>::Block as sp_runtime::traits::Block>::Header:
-			sp_runtime::traits::Header<Number = RelayBlockNumber>,
+		T::BridgedChain:
+			Chain<Hash = RelayBlockHash, Hasher = RelayBlockHasher>,
+		<<T::BridgedChain as Chain>::Block as sp_runtime::traits::Block>::Header: sp_runtime::traits::Header<Number = RelayBlockNumber>
 	{
 		type BridgedRelayChain = T::BridgedChain;
 	}

bridges/primitives/runtime/src/chain.rs

@@ -16,7 +16,7 @@
 
 use codec::{Decode, Encode, MaxEncodedLen};
 use frame_support::{weights::Weight, Parameter};
-use num_traits::{AsPrimitive, Bounded, CheckedSub, SaturatingAdd, Zero};
+use num_traits::{Bounded, CheckedSub, SaturatingAdd, Zero};
 use sp_runtime::{
 	traits::{
 		AtLeast32Bit, AtLeast32BitUnsigned, Block as BlockT, Hash as HashT, Header as HeaderT,
@@ -90,10 +90,7 @@ impl<ChainCall: Encode> Encode for EncodedOrDecodedCall<ChainCall> {
 }
 
 /// Minimal Substrate-based chain representation that may be used from no_std environment.
-pub trait Chain: Send + Sync + 'static
-where
-	<<Self::Block as BlockT>::Header as HeaderT>::Number: AsPrimitive<usize>,
-{
+pub trait Chain: Send + Sync + 'static {
 	/// A type that fulfills the abstract idea of what a Substrate hash is.
 	// Constraits come from the associated Hash type of `sp_runtime::traits::Header`
 	// See here for more info:
@@ -121,7 +118,8 @@ where
 	/// A type that fulfills the abstract idea of what a Substrate block is.
 	// See here for more info:
 	// https://crates.parity.io/sp_runtime/traits/trait.Block.html
-	type Block: Parameter + BlockT<Hash = Self::Hash> + MaybeSerialize;
+	type Block: Parameter + BlockT<Hash = Self::Hash> + MaybeSerialize
+		where Block::Header::Number: AsPrimitive<usize>;
 
 	/// The user account identifier type for the runtime.
 	type AccountId: Parameter
@@ -170,18 +168,14 @@ where
 }
 
 /// A trait that provides the type of the underlying chain.
-pub trait UnderlyingChainProvider
-where
-	<<<Self::Chain as Chain>::Block as BlockT>::Header as HeaderT>::Number: AsPrimitive<usize>,
-{
+pub trait UnderlyingChainProvider {
 	/// Underlying chain type.
 	type Chain: Chain;
 }
 
 impl<T> Chain for T
 where
 	T: Send + Sync + 'static + UnderlyingChainProvider,
-	<<<T::Chain as Chain>::Block as BlockT>::Header as HeaderT>::Number: AsPrimitive<usize>,
 {
 	type Hash = <T::Chain as Chain>::Hash;
 	type Hasher = <T::Chain as Chain>::Hasher;
@@ -201,10 +195,7 @@ where
 }
 
 /// Minimal parachain representation that may be used from no_std environment.
-pub trait Parachain: Chain
-where
-	<<<Self as Chain>::Block as BlockT>::Header as HeaderT>::Number: AsPrimitive<usize>,
-{
+pub trait Parachain: Chain {
 	/// Parachain identifier.
 	const PARACHAIN_ID: u32;
 }
@@ -213,18 +204,13 @@ impl<T> Parachain for T
 where
 	T: Chain + UnderlyingChainProvider,
 	<T as UnderlyingChainProvider>::Chain: Parachain,
-	<<<T as Chain>::Block as BlockT>::Header as HeaderT>::Number: AsPrimitive<usize>,
-	<<<T::Chain as Chain>::Block as BlockT>::Header as HeaderT>::Number: AsPrimitive<usize>,
 {
 	const PARACHAIN_ID: u32 = <<T as UnderlyingChainProvider>::Chain as Parachain>::PARACHAIN_ID;
 }
 
 /// Adapter for `Get<u32>` to access `PARACHAIN_ID` from `trait Parachain`
 pub struct ParachainIdOf<Para>(sp_std::marker::PhantomData<Para>);
-impl<Para: Parachain> frame_support::traits::Get<u32> for ParachainIdOf<Para>
-where
-	<<<Para as Chain>::Block as BlockT>::Header as HeaderT>::Number: AsPrimitive<usize>,
-{
+impl<Para: Parachain> frame_support::traits::Get<u32> for ParachainIdOf<Para> {
 	fn get() -> u32 {
 		Para::PARACHAIN_ID
 	}

pallets/parachain-system/src/tests.rs

@@ -29,7 +29,8 @@ use frame_support::{
 	traits::{OnFinalize, OnInitialize},
 	weights::Weight,
 };
-use frame_system::{pallet_prelude::BlockNumberFor, RawOrigin};
+use frame_system::RawOrigin;
+use frame_system::pallet_prelude::BlockNumberFor;
 use hex_literal::hex;
 use relay_chain::HrmpChannelId;
 use sp_core::{blake2_256, H256};

parachains/runtimes/assets/asset-hub-kusama/src/lib.rs

@@ -67,8 +67,8 @@ use pallet_nfts::PalletFeatures;
 pub use parachains_common as common;
 use parachains_common::{
 	impls::{AssetsToBlockAuthor, DealWithFees},
-	AccountId, AssetIdForTrustBackedAssets, AuraId, Balance, BlockNumber, Hash, Header, Index,
-	Signature, AVERAGE_ON_INITIALIZE_RATIO, DAYS, HOURS, MAXIMUM_BLOCK_WEIGHT,
+	AccountId, AssetIdForTrustBackedAssets, AuraId, Balance, BlockNumber, Hash, Header,
+	Index, Signature, AVERAGE_ON_INITIALIZE_RATIO, DAYS, HOURS, MAXIMUM_BLOCK_WEIGHT,
 	NORMAL_DISPATCH_RATIO, SLOT_DURATION,
 };
 use xcm_config::{

parachains/runtimes/assets/asset-hub-polkadot/src/lib.rs

@@ -102,9 +102,9 @@ use pallet_nfts::PalletFeatures;
 pub use parachains_common as common;
 use parachains_common::{
 	impls::{AssetsToBlockAuthor, DealWithFees},
-	AccountId, AssetHubPolkadotAuraId as AuraId, AssetIdForTrustBackedAssets, Balance, BlockNumber,
-	Hash, Header, Index, Signature, AVERAGE_ON_INITIALIZE_RATIO, DAYS, HOURS, MAXIMUM_BLOCK_WEIGHT,
-	NORMAL_DISPATCH_RATIO, SLOT_DURATION,
+	AccountId, AssetHubPolkadotAuraId as AuraId, AssetIdForTrustBackedAssets, Balance,
+	BlockNumber, Hash, Header, Index, Signature, AVERAGE_ON_INITIALIZE_RATIO, DAYS, HOURS,
+	MAXIMUM_BLOCK_WEIGHT, NORMAL_DISPATCH_RATIO, SLOT_DURATION,
 };
 use xcm_config::{
 	DotLocation, FellowshipLocation, ForeignAssetsConvertedConcreteId, GovernanceLocation,

parachains/runtimes/assets/asset-hub-westend/src/lib.rs

@@ -52,9 +52,9 @@ use pallet_asset_conversion_tx_payment::AssetConversionAdapter;
 use pallet_nfts::PalletFeatures;
 pub use parachains_common as common;
 use parachains_common::{
-	impls::DealWithFees, AccountId, AssetIdForTrustBackedAssets, AuraId, Balance, BlockNumber,
-	Hash, Header, Index, Signature, AVERAGE_ON_INITIALIZE_RATIO, DAYS, HOURS, MAXIMUM_BLOCK_WEIGHT,
-	NORMAL_DISPATCH_RATIO, SLOT_DURATION,
+	impls::DealWithFees, AccountId, AssetIdForTrustBackedAssets, AuraId, Balance,
+	BlockNumber, Hash, Header, Index, Signature, AVERAGE_ON_INITIALIZE_RATIO, DAYS, HOURS,
+	MAXIMUM_BLOCK_WEIGHT, NORMAL_DISPATCH_RATIO, SLOT_DURATION,
 };
 use sp_api::impl_runtime_apis;
 use sp_core::{crypto::KeyTypeId, OpaqueMetadata};

parachains/runtimes/bridge-hubs/test-utils/src/test_cases.rs

@@ -656,7 +656,8 @@ pub fn complex_relay_extrinsic_works<Runtime, XcmConfig, HrmpChannelOpener, GPI,
 		.execute_with(|| {
 			let zero: BlockNumberFor<Runtime> = 0u32.into();
 			let genesis_hash = frame_system::Pallet::<Runtime>::block_hash(zero);
-			let mut header: HeaderFor<Runtime> = bp_test_utils::test_header(1u32.into());
+			let mut header: HeaderFor<Runtime> =
+				bp_test_utils::test_header(1u32.into());
 			header.set_parent_hash(genesis_hash);
 			executive_init_block(&header);
 

parachains/runtimes/collectives/collectives-polkadot/src/lib.rs

@@ -80,9 +80,9 @@ use frame_system::{
 };
 pub use parachains_common as common;
 use parachains_common::{
-	impls::DealWithFees, AccountId, AuraId, Balance, BlockNumber, Hash, Header, Index, Signature,
-	AVERAGE_ON_INITIALIZE_RATIO, DAYS, HOURS, MAXIMUM_BLOCK_WEIGHT, MINUTES, NORMAL_DISPATCH_RATIO,
-	SLOT_DURATION,
+	impls::DealWithFees, AccountId, AuraId, Balance, BlockNumber, Hash, Header, Index,
+	Signature, AVERAGE_ON_INITIALIZE_RATIO, DAYS, HOURS, MAXIMUM_BLOCK_WEIGHT, MINUTES,
+	NORMAL_DISPATCH_RATIO, SLOT_DURATION,
 };
 use xcm_config::{GovernanceLocation, XcmConfig, XcmOriginToTransactDispatchOrigin};