fixing spellcheck, clippy and rustdoc

bin/runtime-common/src/messages_xcm_extension.rs

@@ -207,7 +207,7 @@ impl LocalXcmQueueManager {
 	}
 }
 
-/// A structure that implements [`frame_support:traits::messages::ProcessMessage`] and may
+/// A structure that implements [`frame_support::traits::ProcessMessage`] and may
 /// be used in the `pallet-message-queue` configuration to stop processing messages when the
 /// bridge queue is congested.
 ///
@@ -243,18 +243,19 @@ where
 	) -> Result<bool, ProcessMessageError> {
 		// if the queue is suspended, yield immediately
 		let sender_and_lane = SL::get();
-		if origin.clone().into() == sender_and_lane.location {
-			if LocalXcmQueueManager::is_inbound_queue_suspended::<R, MI>(sender_and_lane.lane) {
-				return Err(ProcessMessageError::Yield)
-			}
+		let is_expected_origin = origin.clone().into() == sender_and_lane.location;
+		if is_expected_origin &&
+			LocalXcmQueueManager::is_inbound_queue_suspended::<R, MI>(sender_and_lane.lane)
+		{
+			return Err(ProcessMessageError::Yield)
 		}
 
 		// else pass message to backed processor
 		Inner::process_message(message, origin, meter, id)
 	}
 }
 
-/// A structure that implements [`frame_support:traits::messages::QueuePausedQuery`] and may
+/// A structure that implements [`frame_support::traits::QueuePausedQuery`] and may
 /// be used in the `pallet-message-queue` configuration to stop processing messages when the
 /// bridge queue is congested.
 ///
@@ -280,10 +281,11 @@ where
 
 		// if we have suspended the queue before, do not even start processing its messages
 		let sender_and_lane = SL::get();
-		if origin.clone().into() == sender_and_lane.location {
-			if LocalXcmQueueManager::is_inbound_queue_suspended::<R, MI>(sender_and_lane.lane) {
-				return true
-			}
+		let is_expected_origin = origin.clone().into() == sender_and_lane.location;
+		if is_expected_origin &&
+			LocalXcmQueueManager::is_inbound_queue_suspended::<R, MI>(sender_and_lane.lane)
+		{
+			return true
 		}
 
 		// else process message

modules/xcm-bridge-hub-router/src/lib.rs

@@ -73,7 +73,7 @@ pub mod pallet {
 		/// The bridged network that this config is for.
 		type BridgedNetworkId: Get<NetworkId>;
 
-		/// Actual message sender (XCMP/DMP) to the sibling bridge hub location.
+		/// Actual message sender (`HRMP` or `DMP`) to the sibling bridge hub location.
 		type ToBridgeHubSender: SendXcm;
 		/// Underlying channel with the sibling bridge hub. It must match the channel, used
 		/// by the `Self::ToBridgeHubSender`.
@@ -343,10 +343,7 @@ mod tests {
 					.into_inner() / FixedU128::DIV +
 					HRMP_FEE;
 			assert_eq!(
-				XcmBridgeHubRouter::validate(&mut Some(dest), &mut Some(xcm.clone()))
-					.unwrap()
-					.1
-					.get(0),
+				XcmBridgeHubRouter::validate(&mut Some(dest), &mut Some(xcm)).unwrap().1.get(0),
 				Some(&(BridgeFeeAsset::get(), expected_fee).into()),
 			);
 		});

modules/xcm-bridge-hub-router/src/mock.rs

@@ -114,7 +114,7 @@ impl SendXcm for TestToBridgeHubSender {
 
 	fn deliver(_ticket: Self::Ticket) -> Result<XcmHash, SendError> {
 		frame_support::storage::unhashed::put(b"TestToBridgeHubSender.Sent", &true);
-		Ok([0u8; 32].into())
+		Ok([0u8; 32])
 	}
 }
 

primitives/messages/src/target_chain.rs

@@ -96,7 +96,7 @@ pub trait MessageDispatch {
 	/// simply drop messages if it returns `false`. The consumer may still call the `dispatch`
 	/// if dispatcher has returned `false`.
 	///
-	/// We check it in the messages delivery transaction prolgoue. So if it becomes `false`
+	/// We check it in the messages delivery transaction prologue. So if it becomes `false`
 	/// after some portion of messages is already dispatched, it doesn't fail the whole transaction.
 	fn is_active() -> bool;