From 027fb5c03e3cc8dc154a0be58ac2b6f5b269b706 Mon Sep 17 00:00:00 2001 From: Kelsi Date: Mon, 9 Feb 2026 18:28:47 -0800 Subject: [PATCH] Always log MWMO parsing results, even when empty Removed condition that suppressed logging when wmoNames.size() == 0. Now will always show 'Loaded X WMO names from MWMO chunk' to confirm MWMO parsing is happening and reveal if ADT tiles have empty or missing WMO name lists. This will help diagnose why cathedral WMOs aren't appearing in logs. --- src/pipeline/adt_loader.cpp | 16 +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/src/pipeline/adt_loader.cpp b/src/pipeline/adt_loader.cpp index 1c066176..295d3ded 100644 --- a/src/pipeline/adt_loader.cpp +++ b/src/pipeline/adt_loader.cpp @@ -203,15 +203,13 @@ void ADTLoader::parseMWMO(const uint8_t* data, size_t size, ADTTerrain& terrain) offset += nameLen + 1; } - if (terrain.wmoNames.size() > 0) { - LOG_INFO("Loaded ", terrain.wmoNames.size(), " WMO names"); - for (size_t i = 0; i < terrain.wmoNames.size(); i++) { - LOG_INFO(" WMO[", i, "]: ", terrain.wmoNames[i]); - // Flag potential duplicate cathedral models - if (terrain.wmoNames[i].find("cathedral") != std::string::npos || - terrain.wmoNames[i].find("Cathedral") != std::string::npos) { - LOG_INFO("*** CATHEDRAL WMO FOUND: ", terrain.wmoNames[i]); - } + LOG_INFO("Loaded ", terrain.wmoNames.size(), " WMO names from MWMO chunk"); + for (size_t i = 0; i < terrain.wmoNames.size(); i++) { + LOG_INFO(" WMO[", i, "]: ", terrain.wmoNames[i]); + // Flag potential duplicate cathedral models + if (terrain.wmoNames[i].find("cathedral") != std::string::npos || + terrain.wmoNames[i].find("Cathedral") != std::string::npos) { + LOG_INFO("*** CATHEDRAL WMO FOUND: ", terrain.wmoNames[i]); } } }