Rename Connection::configuration to config and add some clientbound f…

…unctions that already existed serverbound

taken from Shay's fork: Shays-Forks@b0ca607

azalea-client/src/client.rs

@@ -451,7 +451,7 @@ impl Client {
                         p.game_profile
                     );
                     conn.write(ServerboundLoginAcknowledged {}).await?;
-                    break (conn.configuration(), p.game_profile);
+                    break (conn.config(), p.game_profile);
                 }
                 ClientboundLoginPacket::LoginDisconnect(p) => {
                     debug!("Got disconnect {:?}", p);

azalea-protocol/azalea-protocol-macros/src/lib.rs

@@ -364,7 +364,7 @@ pub fn declare_state_packets(input: TokenStream) -> TokenStream {
                 }
             }
 
-            /// Read a packet by its id, ConnectionProtocol, and flow
+            /// Read a packet by its id, ConnectionProtocol, and flow.
             fn read(
                 id: u32,
                 buf: &mut std::io::Cursor<&[u8]>,
@@ -408,7 +408,7 @@ pub fn declare_state_packets(input: TokenStream) -> TokenStream {
                 }
             }
 
-            /// Read a packet by its id, ConnectionProtocol, and flow
+            /// Read a packet by its id, ConnectionProtocol, and flow.
             fn read(
                 id: u32,
                 buf: &mut std::io::Cursor<&[u8]>,
@@ -422,6 +422,13 @@ pub fn declare_state_packets(input: TokenStream) -> TokenStream {
                 })
             }
         }
+
+        impl crate::packets::Packet<#clientbound_state_name> for #clientbound_state_name {
+            /// No-op, exists so you can pass a packet enum when a Packet<> is expected.
+            fn into_variant(self) -> #clientbound_state_name {
+                self
+            }
+        }
     });
 
     contents.into()

azalea-protocol/src/connect.rs

@@ -359,7 +359,7 @@ impl Connection<ClientboundLoginPacket, ServerboundLoginPacket> {
     /// Change our state from login to configuration. This is the state where
     /// the server sends us the registries and resource pack and stuff.
     #[must_use]
-    pub fn configuration(self) -> Connection<ClientboundConfigPacket, ServerboundConfigPacket> {
+    pub fn config(self) -> Connection<ClientboundConfigPacket, ServerboundConfigPacket> {
         Connection::from(self)
     }
 
@@ -493,7 +493,7 @@ impl Connection<ServerboundLoginPacket, ClientboundLoginPacket> {
 
     /// Change our state back to configuration.
     #[must_use]
-    pub fn configuration(self) -> Connection<ServerboundConfigPacket, ClientboundConfigPacket> {
+    pub fn config(self) -> Connection<ServerboundConfigPacket, ClientboundConfigPacket> {
         Connection::from(self)
     }
 }
@@ -519,7 +519,14 @@ impl Connection<ClientboundConfigPacket, ServerboundConfigPacket> {
 impl Connection<ClientboundGamePacket, ServerboundGamePacket> {
     /// Change our state back to configuration.
     #[must_use]
-    pub fn configuration(self) -> Connection<ClientboundConfigPacket, ServerboundConfigPacket> {
+    pub fn config(self) -> Connection<ClientboundConfigPacket, ServerboundConfigPacket> {
+        Connection::from(self)
+    }
+}
+impl Connection<ServerboundGamePacket, ClientboundGamePacket> {
+    /// Change our state back to configuration.
+    #[must_use]
+    pub fn config(self) -> Connection<ServerboundConfigPacket, ClientboundConfigPacket> {
         Connection::from(self)
     }
 }