Remove unused debug comments and code fragments from rendering logic and shaders for cleaner production-ready code.

Author: gpx1000

attachments/simple_engine/renderer_rendering.cpp

@@ -312,7 +312,6 @@ void Renderer::renderReflectionPass(vk::raii::CommandBuffer                    &
                                     CameraComponent                            *camera,
                                     const std::vector<std::unique_ptr<Entity>> &entities)
 {
-	// Initial scaffolding: clear the reflection RT; drawing the mirrored scene will be added next.
 	if (reflections.empty())
 		return;
 	auto &rt = reflections[currentFrame];
@@ -2395,10 +2394,7 @@ void Renderer::Render(const std::vector<std::unique_ptr<Entity>> &entities, Came
 			// As a last guard before dispatch, make sure compute binding 0 is valid for this frame
 			refreshForwardPlusComputeLightsBindingForFrame(currentFrame);
 
-			// Forward+ per-frame debug printing removed
-
 			dispatchForwardPlus(commandBuffers[currentFrame], tilesX, tilesY, forwardPlusSlicesZ);
-			// Forward+ debug dumps and tile header prints removed
 		}
 
 		// PASS 1: RENDER OPAQUE OBJECTS TO OFF-SCREEN TEXTURE
@@ -2418,7 +2414,6 @@ void Renderer::Render(const std::vector<std::unique_ptr<Entity>> &entities, Came
 		// Clear the off-screen target at the start of opaque rendering to a neutral black background
 		vk::RenderingAttachmentInfo colorAttachment{.imageView = *opaqueSceneColorImageView, .imageLayout = vk::ImageLayout::eColorAttachmentOptimal, .loadOp = vk::AttachmentLoadOp::eClear, .storeOp = vk::AttachmentStoreOp::eStore, .clearValue = vk::ClearColorValue(std::array<float, 4>{0.0f, 0.0f, 0.0f, 1.0f})};
 		depthAttachment.imageView = *depthImageView;
-		// Load depth only if we actually performed a pre-pass (and not in opaque-only debug which intentionally ignores transparency ordering)
 		depthAttachment.loadOp = (didOpaqueDepthPrepass) ? vk::AttachmentLoadOp::eLoad : vk::AttachmentLoadOp::eClear;
 		vk::RenderingInfo passInfo{.renderArea = vk::Rect2D({0, 0}, swapChainExtent), .layerCount = 1, .colorAttachmentCount = 1, .pColorAttachments = &colorAttachment, .pDepthAttachment = &depthAttachment};
 		commandBuffers[currentFrame].beginRendering(passInfo);

attachments/simple_engine/shaders/pbr.slang

@@ -190,8 +190,6 @@ float4 PSMain(VSOutput input) : SV_TARGET
 
     bool forceGlobal = false;
 
-    // No one-frame fragment debug
-
     // Accumulate per-light diffuse and specular terms using GGX microfacet BRDF.
     if (!forceGlobal && count > 0) {
         // Use Forward+ culled list
@@ -235,7 +233,6 @@ float4 PSMain(VSOutput input) : SV_TARGET
                 float3 kD    = (1.0 - F) * (1.0 - metallic);
                 specularLighting += spec * radiance * NdotL;
                 diffuseLighting  += (kD * albedo / PI) * radiance * NdotL;
-                // no debug writes
             }
         }
     }
@@ -279,7 +276,6 @@ float4 PSMain(VSOutput input) : SV_TARGET
         }
     }
 
-    // No debug paths
     float3 ambient = albedo * ao * (0.03 * ubo.scaleIBLAmbient);
     float3 opaqueLit = diffuseLighting + specularLighting + ambient + emissive;
 
@@ -392,7 +388,7 @@ float4 GlassPSMain(VSOutput input) : SV_TARGET
             refl = opaqueSceneColor.Sample(uvR).rgb;
         }
 
-        // Stylized, stable glass: do NOT sample opaqueSceneColor. Use a tinted
+        // Stylized, stable glass: Use a tinted
         // glass body + rim highlight, then add planar reflection contribution.
 
         // Use symmetric |N·V| so that front/back views of thin glass walls
@@ -522,7 +518,6 @@ float4 GlassPSMain(VSOutput input) : SV_TARGET
 
 
     // --- 3. Post-processing (same as PSMain) ---
-    // No debug recording
     color *= ubo.exposure;
 
     // Uncharted2 / Hable filmic tonemap. Use the canonical form without
@@ -538,7 +533,5 @@ float4 GlassPSMain(VSOutput input) : SV_TARGET
         color = saturate(color);
     }
 
-    // (fragment debug disabled in this build)
-
     return float4(color, alphaOut);
 }

attachments/simple_engine/shaders/ray_query.slang

@@ -472,8 +472,6 @@ HitInfo traceRay(float3 origin, float3 direction, float tMin, float tMax) {
         }
     }
 
-    // (No debugPrintf in production)
-    
     // Check if we hit anything
     if (q.CommittedStatus() == COMMITTED_TRIANGLE_HIT) {
         result.hit = true;
@@ -493,8 +491,6 @@ HitInfo traceRay(float3 origin, float3 direction, float tMin, float tMax) {
         uint blasIndex = instIndex;
         uint primitiveIndex = q.CommittedPrimitiveIndex();
 
-        // Minimal debug disabled in production
-        
         // Validate instance index is in bounds of geometry info buffer
         if (blasIndex >= uint(max(0, ubo.geometryInfoCount))) {
             // Invalid BLAS index, use default values
@@ -575,8 +571,6 @@ HitInfo traceRay(float3 origin, float3 direction, float tMin, float tMax) {
         uint i1 = indexBuffer[idxBase + 1u];
         uint i2 = indexBuffer[idxBase + 2u];
 
-        // CRITICAL: Validate vertex indices are within bounds to prevent GPU hang
-        // Reading beyond vertex buffer bounds causes GPU fault/hang
         if (i0 >= geoInfo.vertexCount || i1 >= geoInfo.vertexCount || i2 >= geoInfo.vertexCount) {
             // Out of bounds vertex indices - still present material-derived color
             result.normal = float3(0, 1, 0);
@@ -780,8 +774,6 @@ HitInfo traceRay(float3 origin, float3 direction, float tMin, float tMax) {
         result.alphaMode = material.alphaMode;
         result.opacity = clamp(material.alpha, 0.0, 1.0);
         result.thinWalled = (material.thinWalled != 0);
-
-        // (No debug-only overrides in production)
     }
 
     return result;
@@ -800,8 +792,6 @@ void main(uint3 dispatchThreadID : SV_DispatchThreadID)
         return;
     }
 
-    // (No debug-only heartbeat/screen-test paths in production)
-
     // Generate primary ray
     float2 uv = (float2(pixelCoord) + 0.5) / float2(imageDim);
     float2 ndc = uv * 2.0 - 1.0;
@@ -832,18 +822,13 @@ void main(uint3 dispatchThreadID : SV_DispatchThreadID)
 
     HitInfo hit = traceRay(rayOrigin, rayDir, 0.0001, 10000.0);
 
-    // Production: normal shading (no instance visualization)
-    bool colorByInstance = false;
-
     if (hit.hit) {
         float3 c = shadeWithSecondaryRays(rayOrigin, rayDir, hit);
         // Output linear HDR-ish color; composite pass will apply exposure/gamma.
         outputImage[pixelCoord] = float4(c, 1.0);
 
-        // (Debug print hooks intentionally removed from default path)
     } else {
         // Sky/background color
         outputImage[pixelCoord] = float4(skyColor(rayDir), 1.0);
     }
-    // Debug probes disabled in production
 }