From f19eecb7d359d449c7ede0c7adf6ca37dae6d743 Mon Sep 17 00:00:00 2001
From: Andy Newjack <andynewjack@protonmail.com>
Date: Sat, 25 Jul 2020 22:28:34 -0300
Subject: [PATCH] clean up comments

---
 src/packet/messages.rs | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/src/packet/messages.rs b/src/packet/messages.rs
index 97c9102..872c19f 100644
--- a/src/packet/messages.rs
+++ b/src/packet/messages.rs
@@ -165,7 +165,7 @@ pub struct PlayerTeleported {
 #[pso_message(0x25)]
 pub struct PlayerEquipItem {
     item_id: u32,
-    sub_menu: u8, // changes based on which menu used to equip item. f12 -> item pack -> items = 0, quick and f12 -> item pack -> equip = 6
+    sub_menu: u8,
     unknown1: u16,
 }
 
@@ -419,7 +419,7 @@ pub struct LobbyEmote {
 pub struct RemoveItemFromFloor {
     client_id: u8,
     unknown: u8,
-    map_area: u8, // change to u16 to be consistent with other packets
+    map_area: u8,
     unknown2: u8,
     item_id: u32,
 }
@@ -427,16 +427,14 @@ pub struct RemoveItemFromFloor {
 #[pso_message(0x5A)]
 pub struct PickupItem  {
     item_id: u32,
-    map_area: u8, // change to u16 to be consistent with other packets
+    map_area: u8,
     unknown: [u8; 3],
 }
 
-// this name may be a bit incorrect, it differs from ItemDrop only in that
-// it does not have a y as DropCoordinates only specifies an xz
 #[pso_message(0x5D)]
 pub struct DropSplitStack {
-    pub map_area: u8, // change to u16 to be consistent with other packets
-    pub variety: u8, // what is variety used for? is it ever non-0?
+    pub map_area: u8,
+    pub variety: u8,
     pub unknown1: u16,
     pub x: f32,
     pub z: f32,