Seems the clamp was required

ZDoom · Jan 18, 2020 · 43562c9 · 43562c9
1 parent 9bbd09b
commit 43562c9
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/src/rendering/polyrenderer/drawers/screen_blend.cpp b/src/rendering/polyrenderer/drawers/screen_blend.cpp
@@ -547,6 +547,10 @@ void BlendColorColormap(int y, int x0, int x1, PolyTriangleThreadData* thread)
 		g = (startG + ((gray * rangeG) >> 8)) << 1;
 		b = (startB + ((gray * rangeB) >> 8)) << 1;
 
+		r = MIN(r, (uint32_t)255);
+		g = MIN(g, (uint32_t)255);
+		b = MIN(b, (uint32_t)255);
+
 		line[x] = MAKEARGB(a, r, g, b);
 	}
 }

diff --git a/src/rendering/polyrenderer/drawers/screen_shader.cpp b/src/rendering/polyrenderer/drawers/screen_shader.cpp
@@ -561,6 +561,10 @@ static void MainFP(int x0, int x1, PolyTriangleThreadData* thread)
 				g = (startG + ((gray * rangeG) >> 8)) << 1;
 				b = (startB + ((gray * rangeB) >> 8)) << 1;
 
+				r = MIN(r, (uint32_t)255);
+				g = MIN(g, (uint32_t)255);
+				b = MIN(b, (uint32_t)255);
+
 				fragcolor[x] = MAKEARGB(a, r, g, b);
 			}
 		}