Remove unnecessary generics

Author: ales-otf

node/src/client.rs

@@ -1,76 +1,16 @@
-use scale_codec::Codec;
-// Substrate
+use node_subtensor_runtime::{opaque::Block, RuntimeApi};
 use sc_executor::WasmExecutor;
-use sp_runtime::traits::{Block as BlockT, MaybeDisplay};
-
-use crate::ethereum::EthCompatRuntimeApiCollection;
 
 /// Full backend.
-pub type FullBackend<B> = sc_service::TFullBackend<B>;
+pub type FullBackend = sc_service::TFullBackend<Block>;
 /// Full client.
-pub type FullClient<B, RA, HF> = sc_service::TFullClient<B, RA, WasmExecutor<HF>>;
-
-/// A set of APIs that every runtime must implement.
-pub trait BaseRuntimeApiCollection<Block: BlockT>:
-    sp_api::ApiExt<Block>
-    + sp_api::Metadata<Block>
-    + sp_block_builder::BlockBuilder<Block>
-    + sp_offchain::OffchainWorkerApi<Block>
-    + sp_session::SessionKeys<Block>
-    + sp_transaction_pool::runtime_api::TaggedTransactionQueue<Block>
-{
-}
-
-impl<Block, Api> BaseRuntimeApiCollection<Block> for Api
-where
-    Block: BlockT,
-    Api: sp_api::ApiExt<Block>
-        + sp_api::Metadata<Block>
-        + sp_block_builder::BlockBuilder<Block>
-        + sp_offchain::OffchainWorkerApi<Block>
-        + sp_session::SessionKeys<Block>
-        + sp_transaction_pool::runtime_api::TaggedTransactionQueue<Block>,
-{
-}
-
-/// A set of APIs that Subtensor runtime must implement.
-pub trait RuntimeApiCollection<
-    Block: BlockT,
-    AuraId: Codec,
-    AccountId: Codec,
-    Nonce: Codec,
-    Balance: Codec + MaybeDisplay,
->:
-    BaseRuntimeApiCollection<Block>
-    + EthCompatRuntimeApiCollection<Block>
-    + sp_consensus_aura::AuraApi<Block, AuraId>
-    + sp_consensus_grandpa::GrandpaApi<Block>
-    + frame_system_rpc_runtime_api::AccountNonceApi<Block, AccountId, Nonce>
-    + pallet_transaction_payment_rpc_runtime_api::TransactionPaymentApi<Block, Balance>
-    + subtensor_custom_rpc_runtime_api::DelegateInfoRuntimeApi<Block>
-    + subtensor_custom_rpc_runtime_api::NeuronInfoRuntimeApi<Block>
-    + subtensor_custom_rpc_runtime_api::SubnetInfoRuntimeApi<Block>
-    + subtensor_custom_rpc_runtime_api::SubnetRegistrationRuntimeApi<Block>
-{
-}
-
-impl<Block, AuraId, AccountId, Nonce, Balance, Api>
-    RuntimeApiCollection<Block, AuraId, AccountId, Nonce, Balance> for Api
-where
-    Block: BlockT,
-    AuraId: Codec,
-    AccountId: Codec,
-    Nonce: Codec,
-    Balance: Codec + MaybeDisplay,
-    Api: BaseRuntimeApiCollection<Block>
-        + EthCompatRuntimeApiCollection<Block>
-        + sp_consensus_aura::AuraApi<Block, AuraId>
-        + sp_consensus_grandpa::GrandpaApi<Block>
-        + frame_system_rpc_runtime_api::AccountNonceApi<Block, AccountId, Nonce>
-        + pallet_transaction_payment_rpc_runtime_api::TransactionPaymentApi<Block, Balance>
-        + subtensor_custom_rpc_runtime_api::DelegateInfoRuntimeApi<Block>
-        + subtensor_custom_rpc_runtime_api::NeuronInfoRuntimeApi<Block>
-        + subtensor_custom_rpc_runtime_api::SubnetInfoRuntimeApi<Block>
-        + subtensor_custom_rpc_runtime_api::SubnetRegistrationRuntimeApi<Block>,
-{
-}
+pub type FullClient = sc_service::TFullClient<Block, RuntimeApi, WasmExecutor<HostFunctions>>;
+/// Always enable runtime benchmark host functions, the genesis state
+/// was built with them so we're stuck with them forever.
+///
+/// They're just a noop, never actually get used if the runtime was not compiled with
+/// `runtime-benchmarks`.
+type HostFunctions = (
+    sp_io::SubstrateHostFunctions,
+    frame_benchmarking::benchmarking::HostFunctions,
+);