fix clippy warnings

Author: Xaeroxe

src/errors.rs

@@ -6,7 +6,7 @@ use thiserror::Error;
 /// An error from the Minecraft networking protocol.
 #[derive(Error, Debug)]
 pub enum MinecraftProtocolError {
-    /// VarInt data was invalid according to the spec.
+    /// `VarInt` data was invalid according to the spec.
     #[error("invalid varint data")]
     InvalidVarInt,
 

src/rcon/client.rs

@@ -36,7 +36,7 @@ pub struct RconClient {
 }
 
 impl RconClient {
-    /// Construct an [RconClient] that connects to the given host and port.
+    /// Construct an [`RconClient`] that connects to the given host and port.
     /// Note: to authenticate use the `authenticate` method, this method does not take a password.
     ///
     /// # Arguments
@@ -61,7 +61,7 @@ impl RconClient {
 
     /// Authenticate with the server, with the given password.
     ///
-    /// If authentication fails, this method will return [RconProtocolError::AuthFailed].
+    /// If authentication fails, this method will return [`RconProtocolError::AuthFailed`].
     ///
     /// # Arguments
     /// * `password` - A string slice that holds the RCON password.

src/status/data.rs

@@ -146,7 +146,7 @@ pub struct ChatComponentObject {
     pub extra: Option<Vec<ChatObject>>,
 }
 
-/// ClickEvent data for a chat component
+/// `ClickEvent` data for a chat component
 #[derive(Debug, Serialize, Deserialize)]
 pub struct ChatClickEvent {
     // These are not renamed on purpose. (server returns them in snake_case)
@@ -169,7 +169,7 @@ pub struct ChatClickEvent {
     pub copy_to_clipboard: Option<String>,
 }
 
-/// HoverEvent data for a chat component
+/// `HoverEvent` data for a chat component
 #[derive(Debug, Serialize, Deserialize)]
 pub struct ChatHoverEvent {
     // These are not renamed on purpose. (server returns them in snake_case)

src/varint.rs

@@ -107,7 +107,7 @@ mod tests {
             (i32::MIN, b"\x80\x80\x80\x80\x08"),
         ]);
 
-        for (k, v) in cases.into_iter() {
+        for (k, v) in cases {
             let varint: VarInt = k.into();
             assert_eq!(varint.bytes.len(), v.len());
             assert_eq!(varint.bytes, v);
@@ -140,7 +140,7 @@ mod tests {
         })
         .collect::<HashMap<_, _>>();
 
-        for (k, v) in cases.into_iter() {
+        for (k, v) in cases {
             let x: Result<i32, _> = v.try_into();
 
             if let Err(MinecraftProtocolError::InvalidVarInt) = x {