From 4fbce8572c33c39ec89adf43e50593e0e5626b34 Mon Sep 17 00:00:00 2001
From: avolkov <avolkov@8092fc87-c524-0410-9efc-e669fe64eaf9>
Date: Thu, 17 Dec 2009 04:54:36 +0000
Subject: [PATCH] Cleanup of DrawPlanet() scan tint processing

git-svn-id: svn://svn.code.sf.net/p/sc2/code/trunk@3437 8092fc87-c524-0410-9efc-e669fe64eaf9
---
 sc2/src/uqm/planets/pl_stuff.c | 104 ++++++++++++++++-----------------
 sc2/src/uqm/planets/scan.c     |   6 +-
 2 files changed, 53 insertions(+), 57 deletions(-)

diff --git a/sc2/src/uqm/planets/pl_stuff.c b/sc2/src/uqm/planets/pl_stuff.c
index ca3dd5019..23b6c1073 100644
--- a/sc2/src/uqm/planets/pl_stuff.c
+++ b/sc2/src/uqm/planets/pl_stuff.c
@@ -242,9 +242,38 @@ RotatePlanetSphere (BOOLEAN keepRate)
 	PrepareNextRotationFrame ();
 }
 
+static void
+renderTintFrame (Color tintColor)
+{
+	PLANET_ORBIT *Orbit = &pSolarSysState->Orbit;
+	DWORD pix;
+	int denom;
+	FRAME topoFrame = SetAbsFrameIndex (Orbit->TintFrame, 0);
+	FRAME tintFrame = SetAbsFrameIndex (Orbit->TintFrame, 1);
+
+	// render a solid color frame we will use for tinting
+#ifdef USE_ADDITIVE_SCAN_BLIT
+	tintColor.r /= 2;
+	tintColor.g /= 2;
+	tintColor.b /= 2;
+	denom = -1;
+#else
+	tintColor.a = 128;
+	denom = 0;
+#endif
+	pix = frame_mapRGBA (tintFrame, tintColor.r, tintColor.g, tintColor.b,
+			tintColor.a);
+	fill_frame_rgb (tintFrame, pix, 0, 0, 0, 0);
+
+	// copy topoFrame
+	arith_frame_blit (pSolarSysState->TopoFrame, NULL, topoFrame, NULL, 0, 0);
+	// apply the tint
+	arith_frame_blit (tintFrame, NULL, topoFrame, NULL, 0, denom);
+}
+
 // tintColor.a is ignored
 void
-DrawPlanet (int dy, Color tintColor)
+DrawPlanet (int tintY, Color tintColor)
 {
 	STAMP s;
 	UBYTE a = 128;
@@ -252,73 +281,44 @@ DrawPlanet (int dy, Color tintColor)
 
 	s.origin.x = 0;
 	s.origin.y = 0;
-	s.frame = pSolarSysState->TopoFrame;
+	
 	BatchGraphics ();
 	if (sameColor (tintColor, BLACK_COLOR))
 	{	// no tint -- just draw the surface
+		s.frame = pSolarSysState->TopoFrame;
 		DrawStamp (&s);
 	}
 	else
 	{	// apply different scan type tints
-		UBYTE r, g, b;
-		DWORD p;
-		COUNT framew, frameh;
-		RECT srect, drect, *psrect = NULL, *pdrect = NULL;
-		FRAME tintFrame[2];
-
-		tintFrame[0] = SetAbsFrameIndex (Orbit->TintFrame, 0);
-		tintFrame[1] = SetAbsFrameIndex (Orbit->TintFrame, 1);
-
-		framew = GetFrameWidth (tintFrame[0]);
-		frameh = GetFrameHeight (tintFrame[0]);
+		FRAME tintFrame = SetAbsFrameIndex (Orbit->TintFrame, 0);
+		int height = GetFrameHeight (tintFrame);
 
 		if (!sameColor (tintColor, Orbit->TintColor))
 		{
+			renderTintFrame (tintColor);
 			Orbit->TintColor = tintColor;
-			// Buffer the topoMap to the tintFrame;
-			arith_frame_blit (s.frame, NULL, tintFrame[0], NULL, 0, 0);
-			r = tintColor.r / 2;
-			g = tintColor.g / 2;
-			b = tintColor.b / 2;
-#ifdef USE_ADDITIVE_SCAN_BLIT
-			a = 255;
-#endif
-			p = frame_mapRGBA (tintFrame[1], r, g, b, a);
-			fill_frame_rgb (tintFrame[1], p, 0, 0, 0, 0);
 		}
 		
-		drect.corner.x = 0;
-		drect.extent.width = framew;
-		srect.corner.x = 0;
-		srect.corner.y = 0;
-		srect.extent.width = framew;
-		if (dy >= 0 && dy <= frameh)
-		{
-			drect.corner.y = dy;
-			drect.extent.height = 1;
-			pdrect = &drect;
-			srect.extent.height = 1;
-			psrect = &srect;
-		}
-		if (dy < 0)
-		{
-			drect.corner.y = -dy;
-			drect.extent.height = frameh + dy;
-			pdrect = &drect;
-			srect.extent.height = frameh + dy;
-			psrect = &srect;
+		if (tintY < height - 1)
+		{	// untinted piece showing, draw regular topo
+			s.frame = pSolarSysState->TopoFrame;
+			DrawStamp (&s);
 		}
 
-		if (dy <= frameh)
-		{
-#ifdef USE_ADDITIVE_SCAN_BLIT
-			arith_frame_blit (tintFrame[1], psrect, tintFrame[0], pdrect, 0, -1);
-#else
-			arith_frame_blit (tintFrame[1], psrect, tintFrame[0], pdrect, 0, 0);
-#endif
+		if (tintY >= 0)
+		{	// tinted piece showing, draw tinted piece
+			RECT oldClipRect;
+			RECT clipRect;
+
+			// adjust cliprect to confine the tint
+			GetContextClipRect (&oldClipRect);
+			clipRect = oldClipRect;
+			clipRect.extent.height = tintY + 1;
+			SetContextClipRect (&clipRect);
+			s.frame = tintFrame;
+			DrawStamp (&s);
+			SetContextClipRect (&oldClipRect);
 		}
-		s.frame = tintFrame[0];
-		DrawStamp (&s);
 	}
 	UnbatchGraphics ();
 }
diff --git a/sc2/src/uqm/planets/scan.c b/sc2/src/uqm/planets/scan.c
index 1505bfb01..b8de5e0b3 100644
--- a/sc2/src/uqm/planets/scan.c
+++ b/sc2/src/uqm/planets/scan.c
@@ -1072,15 +1072,11 @@ ScanPlanet (COUNT scanType)
 		{	// Aborted by a keypress; draw in finished state
 			LockMutex (GraphicsLock);
 			BatchGraphics ();
-			// dy < 0 means "from dy to the end"
-			DrawPlanet (-i, tintColor);
+			DrawPlanet (SCAN_LINES - 1, tintColor);
 			DrawScannedStuff (SCAN_LINES - 1, scan);
 			UnbatchGraphics ();
 			UnlockMutex (GraphicsLock);
 		}
-
-		// Reset the last used tint color
-		Orbit->TintColor = BLACK_COLOR;
 	}
 
 	LockMutex (GraphicsLock);