From 0b1f1ec2e30368164854d52f34586fd691d33e34 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]); } } }