phaneron/Kelsidavis-WoWee
1
0
Fork 0
mirror of https://github.com/Kelsidavis/WoWee.git synced 2026-05-03 08:03:50 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

game: downgrade per-item-query LOG_INFO to LOG_DEBUG in game_handler

Browse source 
queryItemInfo and handleItemQueryResponse fire for every item in
inventory, loot windows, vendor lists, and mail — potentially dozens
of times at login or when any container is opened.  Downgrade to
LOG_DEBUG to reduce noise.  Also downgrade useItemById search traces
to LOG_DEBUG; the final warning (item not found) stays at LOG_WARNING.
This commit is contained in:
Kelsi 2026-03-10 04:48:33 -07:00
parent f22845b238
commit dd8c2cbb20
1 changed files with 7 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

13
src/game/game_handler.cpp
View file

@ -10044,7 +10044,7 @@ void GameHandler::queryItemInfo(uint32_t entry, uint64_t guid) {
? packetParsers_->buildItemQuery(entry, queryGuid) ? packetParsers_->buildItemQuery(entry, queryGuid)
: ItemQueryPacket::build(entry, queryGuid); : ItemQueryPacket::build(entry, queryGuid);
socket->send(packet); socket->send(packet);
LOG_INFO("queryItemInfo: entry=", entry, " guid=0x", std::hex, queryGuid, std::dec, LOG_DEBUG("queryItemInfo: entry=", entry, " guid=0x", std::hex, queryGuid, std::dec,
" pending=", pendingItemQueries_.size()); " pending=", pendingItemQueries_.size());
} }
@ -10059,9 +10059,8 @@ void GameHandler::handleItemQueryResponse(network::Packet& packet) {
} }
pendingItemQueries_.erase(data.entry); pendingItemQueries_.erase(data.entry);
LOG_INFO("handleItemQueryResponse: entry=", data.entry, " valid=", data.valid, LOG_DEBUG("handleItemQueryResponse: entry=", data.entry, " name='", data.name,
" name='", data.name, "' displayInfoId=", data.displayInfoId, "' displayInfoId=", data.displayInfoId, " pending=", pendingItemQueries_.size());
" pending=", pendingItemQueries_.size());
if (data.valid) { if (data.valid) {
itemInfoCache_[data.entry] = data; itemInfoCache_[data.entry] = data;
@ -14694,12 +14693,12 @@ void GameHandler::useItemInBag(int bagIndex, int slotIndex) {
void GameHandler::useItemById(uint32_t itemId) { void GameHandler::useItemById(uint32_t itemId) {
if (itemId == 0) return; if (itemId == 0) return;
LOG_INFO("useItemById: searching for itemId=", itemId, " in backpack (", inventory.getBackpackSize(), " slots)"); LOG_DEBUG("useItemById: searching for itemId=", itemId);
// Search backpack first // Search backpack first
for (int i = 0; i < inventory.getBackpackSize(); i++) { for (int i = 0; i < inventory.getBackpackSize(); i++) {
const auto& slot = inventory.getBackpackSlot(i); const auto& slot = inventory.getBackpackSlot(i);
if (!slot.empty() && slot.item.itemId == itemId) { if (!slot.empty() && slot.item.itemId == itemId) {
LOG_INFO("useItemById: found itemId=", itemId, " at backpack slot ", i); LOG_DEBUG("useItemById: found itemId=", itemId, " at backpack slot ", i);
useItemBySlot(i); useItemBySlot(i);
return; return;
} }
@ -14710,7 +14709,7 @@ void GameHandler::useItemById(uint32_t itemId) {
for (int slot = 0; slot < bagSize; slot++) { for (int slot = 0; slot < bagSize; slot++) {
const auto& bagSlot = inventory.getBagSlot(bag, slot); const auto& bagSlot = inventory.getBagSlot(bag, slot);
if (!bagSlot.empty() && bagSlot.item.itemId == itemId) { if (!bagSlot.empty() && bagSlot.item.itemId == itemId) {
LOG_INFO("useItemById: found itemId=", itemId, " in bag ", bag, " slot ", slot); LOG_DEBUG("useItemById: found itemId=", itemId, " in bag ", bag, " slot ", slot);
useItemInBag(bag, slot); useItemInBag(bag, slot);
return; return;
} }