Remove all debug messages from rendering and map systems

Co-authored-by: djeada <[email protected]>

app/core/game_engine.cpp

@@ -1407,8 +1407,6 @@ void GameEngine::restoreEnvironmentFromMetadata(const QJsonObject &metadata) {
         m_terrain->configure(*heightMap, terrainService.biomeSettings());
       }
       if (m_river) {
-        qDebug() << "GameEngine: Configuring river renderer with"
-                 << heightMap->getRiverSegments().size() << "segments";
         m_river->configure(heightMap->getRiverSegments(),
                            heightMap->getTileSize());
       }

game/map/map_loader.cpp

@@ -263,7 +263,6 @@ static void readTerrain(const QJsonArray &arr, std::vector<TerrainFeature> &out,
 
 static void readRivers(const QJsonArray &arr, std::vector<RiverSegment> &out,
                        const GridDefinition &grid, CoordSystem coordSys) {
-  qDebug() << "readRivers: Processing" << arr.size() << "river segments";
   out.clear();
   out.reserve(arr.size());
 
@@ -309,16 +308,12 @@ static void readRivers(const QJsonArray &arr, std::vector<RiverSegment> &out,
       segment.width = float(o.value("width").toDouble(2.0));
     }
 
-    qDebug() << "  River segment: start=" << segment.start
-             << "end=" << segment.end << "width=" << segment.width;
     out.push_back(segment);
   }
-  qDebug() << "readRivers: Loaded" << out.size() << "river segments";
 }
 
 static void readBridges(const QJsonArray &arr, std::vector<Bridge> &out,
                         const GridDefinition &grid, CoordSystem coordSys) {
-  qDebug() << "readBridges: Processing" << arr.size() << "bridges";
   out.clear();
   out.reserve(arr.size());
 
@@ -368,11 +363,8 @@ static void readBridges(const QJsonArray &arr, std::vector<Bridge> &out,
       bridge.height = float(o.value("height").toDouble(0.5));
     }
 
-    qDebug() << "  Bridge: start=" << bridge.start << "end=" << bridge.end
-             << "width=" << bridge.width;
     out.push_back(bridge);
   }
-  qDebug() << "readBridges: Loaded" << out.size() << "bridges";
 }
 
 bool MapLoader::loadFromJsonFile(const QString &path, MapDefinition &outMap,

game/map/skirmish_loader.cpp

@@ -241,9 +241,6 @@ SkirmishLoadResult SkirmishLoader::start(const QString &mapPath,
 
   if (m_river) {
     if (terrainService.isInitialized() && terrainService.getHeightMap()) {
-      qDebug() << "SkirmishLoader: Configuring river renderer with"
-               << terrainService.getHeightMap()->getRiverSegments().size()
-               << "segments";
       m_river->configure(terrainService.getHeightMap()->getRiverSegments(),
                          terrainService.getHeightMap()->getTileSize());
     }
@@ -251,9 +248,6 @@ SkirmishLoadResult SkirmishLoader::start(const QString &mapPath,
 
   if (m_bridge) {
     if (terrainService.isInitialized() && terrainService.getHeightMap()) {
-      qDebug() << "SkirmishLoader: Configuring bridge renderer with"
-               << terrainService.getHeightMap()->getBridges().size()
-               << "bridges";
       m_bridge->configure(terrainService.getHeightMap()->getBridges(),
                           terrainService.getHeightMap()->getTileSize());
     }

game/map/terrain.cpp

@@ -516,7 +516,6 @@ void TerrainHeightMap::addRiverSegments(
 
 void TerrainHeightMap::addBridges(const std::vector<Bridge> &bridges) {
   m_bridges = bridges;
-  qDebug() << "TerrainHeightMap: Added" << bridges.size() << "bridges";
 
   const float gridHalfWidth = m_width * 0.5f - 0.5f;
   const float gridHalfHeight = m_height * 0.5f - 0.5f;
@@ -576,8 +575,6 @@ void TerrainHeightMap::addBridges(const std::vector<Bridge> &bridges) {
       }
     }
   }
-
-  qDebug() << "TerrainHeightMap: Bridges configured - areas are now walkable";
 }
 
 } // namespace Game::Map

game/map/terrain_service.cpp

@@ -16,11 +16,7 @@ void TerrainService::initialize(const MapDefinition &mapDef) {
       mapDef.grid.width, mapDef.grid.height, mapDef.grid.tileSize);
 
   m_heightMap->buildFromFeatures(mapDef.terrain);
-  qDebug() << "TerrainService: Adding" << mapDef.rivers.size()
-           << "river segments to height map";
   m_heightMap->addRiverSegments(mapDef.rivers);
-  qDebug() << "TerrainService: Adding" << mapDef.bridges.size()
-           << "bridges to height map";
   m_heightMap->addBridges(mapDef.bridges);
   m_biomeSettings = mapDef.biome;
   m_heightMap->applyBiomeVariation(m_biomeSettings);

game/systems/capture_system.cpp

@@ -63,10 +63,6 @@ void CaptureSystem::transferBarrackOwnership(Engine::Core::World *world,
   int previousOwnerId = unit->ownerId;
   unit->ownerId = newOwnerId;
 
-  std::cout << "[Capture] Barrack " << barrack->getId()
-            << " captured! Previous owner: " << previousOwnerId
-            << ", New owner: " << newOwnerId << std::endl;
-
   QVector3D tc = Game::Visuals::teamColorForOwner(newOwnerId);
   renderable->color[0] = tc.x();
   renderable->color[1] = tc.y();

game/systems/global_stats_registry.cpp

@@ -125,10 +125,6 @@ void GlobalStatsRegistry::onBarrackCaptured(
 
   auto &newStats = m_playerStats[event.newOwnerId];
   newStats.barracksOwned++;
-
-  qDebug() << "[Stats] Barrack captured - Previous owner"
-           << event.previousOwnerId << "lost barrack, new owner"
-           << event.newOwnerId << "gained barrack";
 }
 
 void GlobalStatsRegistry::rebuildFromWorld(Engine::Core::World &world) {

render/ground/bridge_renderer.cpp

@@ -18,20 +18,15 @@ void BridgeRenderer::configure(const std::vector<Game::Map::Bridge> &bridges,
                                float tileSize) {
   m_bridges = bridges;
   m_tileSize = tileSize;
-  qDebug() << "BridgeRenderer::configure() called with" << bridges.size()
-           << "bridges, tileSize:" << tileSize;
   buildMeshes();
 }
 
 void BridgeRenderer::buildMeshes() {
   if (m_bridges.empty()) {
-    qDebug() << "BridgeRenderer::buildMeshes() - No bridges to build";
     m_mesh.reset();
     return;
   }
 
-  qDebug() << "BridgeRenderer::buildMeshes() - Building meshes for"
-           << m_bridges.size() << "bridges";
   std::vector<Vertex> vertices;
   std::vector<unsigned int> indices;
 
@@ -160,13 +155,10 @@ void BridgeRenderer::buildMeshes() {
   }
 
   if (vertices.empty() || indices.empty()) {
-    qDebug() << "BridgeRenderer::buildMeshes() - No vertices/indices generated";
     m_mesh.reset();
     return;
   }
 
-  qDebug() << "BridgeRenderer::buildMeshes() - Created mesh with"
-           << vertices.size() << "vertices and" << indices.size() << "indices";
   m_mesh = std::make_unique<Mesh>(vertices, indices);
 }
 
@@ -213,11 +205,8 @@ void BridgeRenderer::submit(Renderer &renderer, ResourceManager *resources) {
 
   auto shader = renderer.getShader("bridge");
   if (!shader) {
-    qDebug() << "BridgeRenderer::submit() - Bridge shader not found! Falling "
-                "back to basic shader";
     shader = renderer.getShader("basic");
     if (!shader) {
-      qDebug() << "BridgeRenderer::submit() - Basic shader also not found!";
       return;
     }
   }

render/ground/plant_renderer.cpp

@@ -126,11 +126,8 @@ void PlantRenderer::generatePlantInstances() {
 
   const float plantDensity =
       std::clamp(m_biomeSettings.plantDensity, 0.0f, 2.0f);
-  qDebug() << "PlantRenderer: plantDensity =" << plantDensity
-           << "from biome settings";
 
   if (plantDensity < 0.01f) {
-    qDebug() << "PlantRenderer: plantDensity too low, skipping generation";
     m_plantInstanceCount = 0;
     m_plantInstancesDirty = false;
     return;

render/ground/river_renderer.cpp

@@ -16,20 +16,15 @@ void RiverRenderer::configure(
     const std::vector<Game::Map::RiverSegment> &riverSegments, float tileSize) {
   m_riverSegments = riverSegments;
   m_tileSize = tileSize;
-  qDebug() << "RiverRenderer::configure() called with" << riverSegments.size()
-           << "segments, tileSize:" << tileSize;
   buildMeshes();
 }
 
 void RiverRenderer::buildMeshes() {
   if (m_riverSegments.empty()) {
-    qDebug() << "RiverRenderer::buildMeshes() - No river segments to build";
     m_mesh.reset();
     return;
   }
 
-  qDebug() << "RiverRenderer::buildMeshes() - Building meshes for"
-           << m_riverSegments.size() << "river segments";
   std::vector<Vertex> vertices;
   std::vector<unsigned int> indices;
 
@@ -143,19 +138,15 @@ void RiverRenderer::buildMeshes() {
   }
 
   if (vertices.empty() || indices.empty()) {
-    qDebug() << "RiverRenderer::buildMeshes() - No vertices/indices generated";
     m_mesh.reset();
     return;
   }
 
-  qDebug() << "RiverRenderer::buildMeshes() - Created mesh with"
-           << vertices.size() << "vertices and" << indices.size() << "indices";
   m_mesh = std::make_unique<Mesh>(vertices, indices);
 }
 
 void RiverRenderer::submit(Renderer &renderer, ResourceManager *resources) {
   if (!m_mesh || m_riverSegments.empty()) {
-    qDebug() << "RiverRenderer::submit() - No mesh or empty segments, skipping";
     return;
   }
 
@@ -163,7 +154,6 @@ void RiverRenderer::submit(Renderer &renderer, ResourceManager *resources) {
 
   auto shader = renderer.getShader("river");
   if (!shader) {
-    qDebug() << "RiverRenderer::submit() - River shader not found!";
     return;
   }