[Scummvm-cvs-logs] SF.net SVN: scummvm: [29843] scummvm/trunk/engines/cine/gfx.cpp

[thebluegr at users.sourceforge.net]

Revision: 29843
          http://scummvm.svn.sourceforge.net/scummvm/?rev=29843&view=rev
Author:   thebluegr
Date:     2007-12-13 03:11:38 -0800 (Thu, 13 Dec 2007)

Log Message:
-----------
Cleanup

Modified Paths:
--------------
    scummvm/trunk/engines/cine/gfx.cpp

Modified: scummvm/trunk/engines/cine/gfx.cpp
===================================================================
--- scummvm/trunk/engines/cine/gfx.cpp	2007-12-13 10:47:14 UTC (rev 29842)
+++ scummvm/trunk/engines/cine/gfx.cpp	2007-12-13 11:11:38 UTC (rev 29843)
@@ -133,45 +133,32 @@
 	}
 }
 
-static uint16 transformColor(uint16 baseColor, int8 r, int8 g, int8 b) {
-	int8 oriR = (baseColor & 0x7);
-	int8 oriG = (baseColor & 0x70) >> 4;
-	int8 oriB = (baseColor & 0x700) >> 8;
+int8 clipColor(int8 color) {
+	if (color < 0)
+		color = 0;
+	else if (color > 7)
+		color = 7;
 
-	oriR += r;
-	oriG += g;
-	oriB += b;
+	return color;
+}
 
-	if (oriR < 0)
-		oriR = 0;
-	else if (oriR > 7)
-		oriR = 7;
+static uint16 transformColor(uint16 baseColor, int8 r, int8 g, int8 b) {
+	int8 oriR = clipColor((baseColor & 0x7) + r);
+	int8 oriG = clipColor(((baseColor & 0x70) >> 4) + g);
+	int8 oriB = clipColor(((baseColor & 0x700) >> 8) + b);
 
-	if (oriG < 0)
-		oriG = 0;
-	else if (oriG > 7)
-		oriG = 7;
-
-	if (oriB < 0)
-		oriB = 0;
-	else if (oriB > 7)
-		oriB = 7;
-
 	return oriR | (oriG << 4) | (oriB << 8);
 }
 
 void transformPaletteRange(byte startColor, byte stopColor, int8 r, int8 g, int8 b) {
-	byte i;
-
-	for (i = startColor; i <= stopColor; i++) {
+	for (byte i = startColor; i <= stopColor; i++) {
 		c_palette[i] = transformColor(tempPalette[i], b, g, r);
 	}
 	//gfxFlipPage(page2);
 }
 
 void gfxFillSprite(byte *spritePtr, uint16 width, uint16 height, byte *page, int16 x, int16 y, uint8 fillColor) {
-	int16 i;
-	int16 j;
+	int16 i, j;
 
 	for (i = 0; i < height; i++) {
 		byte *destPtr = page + x + y * 320;
@@ -353,9 +340,10 @@
 
 void gfxFlipRawPage(byte *frontBuffer) {
 	byte *page = frontBuffer;
-	int x, y;
+	int x, y, i;
 	byte *pixels = (byte *) screenBuffer;
 	byte c;
+	byte pal[256 * 4];
 
 	for (y = 0; y < 200; y++) {
 		for (x = 0; x < 320; x++) {
@@ -369,9 +357,6 @@
 		}
 	}
 
-	byte pal[256 * 4];
-	int i;
-
 	if (colorMode256) {
 		for (i = 0; i < 256; i++) {
 			pal[i * 4 + 0] = palette256[i * 3 + 0];
@@ -396,8 +381,7 @@
 
 void drawSpriteRaw(byte *spritePtr, byte *maskPtr, int16 width, int16 height,
 				   byte *page, int16 x, int16 y) {
-	int16 i;
-	int16 j;
+	int16 i, j;
 
 	// FIXME: Is it a bug if maskPtr == NULL?
 	if (!maskPtr)
@@ -424,8 +408,7 @@
 
 void drawSpriteRaw2(byte *spritePtr, byte transColor, int16 width, int16 height,
 					byte *page, int16 x, int16 y) {
-	int16 i;
-	int16 j;
+	int16 i, j;
 
 	for (i = 0; i < height; i++) {
 		byte *destPtr = page + x + y * 320;


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.