From 1bfa2fef834bf52155b25e79bc4742363ae1ceac Mon Sep 17 00:00:00 2001 From: Andrei Karas Date: Thu, 23 Oct 2014 12:43:12 +0300 Subject: Add packet fields comments in inventoryhandler. --- src/net/tmwa/inventoryhandler.cpp | 106 +++++++++++++++++++------------------- 1 file changed, 54 insertions(+), 52 deletions(-) (limited to 'src/net/tmwa/inventoryhandler.cpp') diff --git a/src/net/tmwa/inventoryhandler.cpp b/src/net/tmwa/inventoryhandler.cpp index 79d60e381..dec66396e 100644 --- a/src/net/tmwa/inventoryhandler.cpp +++ b/src/net/tmwa/inventoryhandler.cpp @@ -159,8 +159,8 @@ void InventoryHandler::equipItem(const Item *const item) const createOutPacket(CMSG_PLAYER_EQUIP); outMsg.writeInt16(static_cast( - item->getInvIndex() + INVENTORY_OFFSET)); - outMsg.writeInt16(0); + item->getInvIndex() + INVENTORY_OFFSET), "index"); + outMsg.writeInt16(0, "unused"); } void InventoryHandler::unequipItem(const Item *const item) const @@ -170,7 +170,7 @@ void InventoryHandler::unequipItem(const Item *const item) const createOutPacket(CMSG_PLAYER_UNEQUIP); outMsg.writeInt16(static_cast( - item->getInvIndex() + INVENTORY_OFFSET)); + item->getInvIndex() + INVENTORY_OFFSET), "index"); } void InventoryHandler::useItem(const Item *const item) const @@ -180,8 +180,8 @@ void InventoryHandler::useItem(const Item *const item) const createOutPacket(CMSG_PLAYER_INVENTORY_USE); outMsg.writeInt16(static_cast( - item->getInvIndex() + INVENTORY_OFFSET)); - outMsg.writeInt32(item->getId()); // unused + item->getInvIndex() + INVENTORY_OFFSET), "index"); + outMsg.writeInt32(item->getId(), "item id"); } void InventoryHandler::dropItem(const Item *const item, const int amount) const @@ -191,8 +191,8 @@ void InventoryHandler::dropItem(const Item *const item, const int amount) const createOutPacket(CMSG_PLAYER_INVENTORY_DROP); outMsg.writeInt16(static_cast( - item->getInvIndex() + INVENTORY_OFFSET)); - outMsg.writeInt16(static_cast(amount)); + item->getInvIndex() + INVENTORY_OFFSET), "index"); + outMsg.writeInt16(static_cast(amount), "amount"); } void InventoryHandler::closeStorage(const int type A_UNUSED) const @@ -206,15 +206,17 @@ void InventoryHandler::moveItem2(const int source, const int slot, if (source == Inventory::INVENTORY && destination == Inventory::STORAGE) { createOutPacket(CMSG_MOVE_TO_STORAGE); - outMsg.writeInt16(static_cast(slot + INVENTORY_OFFSET)); - outMsg.writeInt32(amount); + outMsg.writeInt16(static_cast(slot + INVENTORY_OFFSET), + "index"); + outMsg.writeInt32(amount, "amount"); } else if (source == Inventory::STORAGE && destination == Inventory::INVENTORY) { createOutPacket(CSMG_MOVE_FROM_STORAGE); - outMsg.writeInt16(static_cast(slot + STORAGE_OFFSET)); - outMsg.writeInt32(amount); + outMsg.writeInt16(static_cast(slot + STORAGE_OFFSET), + "index"); + outMsg.writeInt32(amount, "amount"); } } @@ -238,7 +240,7 @@ void InventoryHandler::processPlayerEquipment(Net::MessageIn &msg) Inventory *const inventory = localPlayer ? PlayerInfo::getInventory() : nullptr; - msg.readInt16(); // length + msg.readInt16("len"); Equipment *const equipment = PlayerInfo::getEquipment(); if (equipment && !equipment->getBackend()) { // look like SMSG_PLAYER_INVENTORY was not received @@ -249,15 +251,15 @@ void InventoryHandler::processPlayerEquipment(Net::MessageIn &msg) for (int loop = 0; loop < number; loop++) { - const int index = msg.readInt16() - INVENTORY_OFFSET; - const int itemId = msg.readInt16(); - const uint8_t itemType = msg.readUInt8(); // type - uint8_t identified = msg.readUInt8(); // identify flag - - msg.readInt16(); // equip type - const int equipType = msg.readInt16(); - msg.readUInt8(); // attribute - const uint8_t refine = msg.readUInt8(); + const int index = msg.readInt16("index") - INVENTORY_OFFSET; + const int itemId = msg.readInt16("item id"); + const uint8_t itemType = msg.readUInt8("item type"); + uint8_t identified = msg.readUInt8("identify"); + + msg.readInt16("equip type?"); + const int equipType = msg.readInt16("equip type"); + msg.readUInt8("attribute"); + const uint8_t refine = msg.readUInt8("refine"); int cards[4]; for (int f = 0; f < 4; f++) cards[f] = msg.readInt16("card"); @@ -301,20 +303,20 @@ void InventoryHandler::processPlayerInventoryAdd(Net::MessageIn &msg) mEquips.clear(); PlayerInfo::getEquipment()->setBackend(&mEquips); } - const int index = msg.readInt16() - INVENTORY_OFFSET; - int amount = msg.readInt16(); - const int itemId = msg.readInt16(); - uint8_t identified = msg.readUInt8(); - msg.readUInt8(); // attribute - const uint8_t refine = msg.readUInt8(); + const int index = msg.readInt16("index") - INVENTORY_OFFSET; + int amount = msg.readInt16("amount"); + const int itemId = msg.readInt16("item id"); + uint8_t identified = msg.readUInt8("identified"); + msg.readUInt8("attribute"); + const uint8_t refine = msg.readUInt8("refine"); int cards[4]; for (int f = 0; f < 4; f++) cards[f] = msg.readInt16("card"); - const int equipType = msg.readInt16(); - msg.readUInt8(); // itemType + const int equipType = msg.readInt16("equip type"); + msg.readUInt8("item type"); const ItemInfo &itemInfo = ItemDB::get(itemId); - const unsigned char err = msg.readUInt8(); + const unsigned char err = msg.readUInt8("status"); int floorId; if (mSentPickups.empty()) { @@ -494,9 +496,9 @@ void InventoryHandler::processPlayerStorage(Net::MessageIn &msg) void InventoryHandler::processPlayerEquip(Net::MessageIn &msg) { BLOCK_START("InventoryHandler::processPlayerEquip") - const int index = msg.readInt16() - INVENTORY_OFFSET; - const int equipType = msg.readInt16(); - const uint8_t flag = msg.readUInt8(); + const int index = msg.readInt16("index") - INVENTORY_OFFSET; + const int equipType = msg.readInt16("equip type"); + const uint8_t flag = msg.readUInt8("flag"); if (!flag) NotifyManager::notify(NotifyTypes::EQUIP_FAILED); @@ -508,9 +510,9 @@ void InventoryHandler::processPlayerEquip(Net::MessageIn &msg) void InventoryHandler::processPlayerUnEquip(Net::MessageIn &msg) { BLOCK_START("InventoryHandler::processPlayerUnEquip") - msg.readInt16(); // inder val - INVENTORY_OFFSET; - const int equipType = msg.readInt16(); - const uint8_t flag = msg.readUInt8(); + msg.readInt16("index"); + const int equipType = msg.readInt16("equip type"); + const uint8_t flag = msg.readUInt8("flag"); if (flag) mEquips.setEquipment(getSlot(equipType), -1); @@ -522,23 +524,23 @@ void InventoryHandler::processPlayerUnEquip(Net::MessageIn &msg) void InventoryHandler::processPlayerStorageEquip(Net::MessageIn &msg) { BLOCK_START("InventoryHandler::processPlayerStorageEquip") - msg.readInt16(); // length + msg.readInt16("len"); const int number = (msg.getLength() - 4) / 20; for (int loop = 0; loop < number; loop++) { int cards[4]; - const int index = msg.readInt16() - STORAGE_OFFSET; - const int itemId = msg.readInt16(); - const uint8_t itemType = msg.readUInt8(); - uint8_t identified = msg.readUInt8(); + const int index = msg.readInt16("index") - STORAGE_OFFSET; + const int itemId = msg.readInt16("item id"); + const uint8_t itemType = msg.readUInt8("item type"); + uint8_t identified = msg.readUInt8("identified"); const int amount = 1; - msg.readInt16(); // Equip Point? - msg.readInt16(); // Another Equip Point? - msg.readUInt8(); // Attribute (broken) - const uint8_t refine = msg.readUInt8(); + msg.readInt16("equip point?"); + msg.readInt16("another equip point?"); + msg.readUInt8("attribute (broken)"); + const uint8_t refine = msg.readUInt8("refine"); for (int i = 0; i < 4; i++) - cards[i] = msg.readInt16(); + cards[i] = msg.readInt16("card"); if (mDebugInventory) { @@ -570,12 +572,12 @@ void InventoryHandler::processPlayerStorageAdd(Net::MessageIn &msg) { BLOCK_START("InventoryHandler::processPlayerStorageAdd") // Move an item into storage - const int index = msg.readInt16() - STORAGE_OFFSET; - const int amount = msg.readInt32(); - const int itemId = msg.readInt16(); - unsigned char identified = msg.readUInt8(); - msg.readUInt8(); // attribute - const uint8_t refine = msg.readUInt8(); + const int index = msg.readInt16("index") - STORAGE_OFFSET; + const int amount = msg.readInt32("amount"); + const int itemId = msg.readInt16("item id"); + unsigned char identified = msg.readUInt8("identified"); + msg.readUInt8("attribute"); + const uint8_t refine = msg.readUInt8("refine"); int cards[4]; for (int f = 0; f < 4; f++) cards[f] = msg.readInt16("card"); -- cgit v1.2.3-70-g09d2