Merge pull request 'remove unknown trade state' (#14) from trade_unknown into master

Reviewed-on: #14
2021-12-09 02:43:28 -05:00 · 2021-12-09 02:43:28 -05:00 · fdd4aa916d
commit fdd4aa916d
parent 564a1ce165 519344e9c4
1 changed files with 0 additions and 10 deletions
--- a/src/packet/messages.rs
+++ b/src/packet/messages.rs
@ -700,7 +700,6 @@ pub enum TradeRequestCommand {
    RemoveItem(u32, u32),
    Confirm,
    FinalConfirm,
-    Unknown,
 }

 impl PSOPacketData for TradeRequestCommand {
@ -738,9 +737,6 @@ impl PSOPacketData for TradeRequestCommand {
            4 => {
                Ok(TradeRequestCommand::FinalConfirm)
            },
-            6 => {
-                Ok(TradeRequestCommand::Unknown)
-            },
            _ => {
                Err(PacketParseError::InvalidValue)
            },
@ -788,12 +784,6 @@ impl PSOPacketData for TradeRequestCommand {
                    .chain(std::iter::repeat(0).take(11))
                    .collect()
            },
-            TradeRequestCommand::Unknown => {
-                vec![6u8]
-                    .into_iter()
-                    .chain(std::iter::repeat(0).take(11))
-                    .collect()
-            },
        }
    }
 }