[Scummvm-cvs-logs] CVS: scummvm/scumm script_v8.cpp,2.86,2.87

[Max Horn]

Update of /cvsroot/scummvm/scummvm/scumm
In directory sc8-pr-cvs1:/tmp/cvs-serv5215

Modified Files:
	script_v8.cpp 
Log Message:
added some comment

Index: script_v8.cpp
===================================================================
RCS file: /cvsroot/scummvm/scummvm/scumm/script_v8.cpp,v
retrieving revision 2.86
retrieving revision 2.87
diff -u -d -r2.86 -r2.87
--- script_v8.cpp	31 Dec 2002 17:34:04 -0000	2.86
+++ script_v8.cpp	31 Dec 2002 18:21:01 -0000	2.87
@@ -957,7 +957,17 @@
 		warning("V8 Load game opcode not implemented");
 		break;
 	case 0x5F:		// SO_ROOM_SATURATION Set saturation of room colors
-		warning("o8_roomOps: SO_ROOM_SATURATION(%d, %d, %d, %d, %d)", pop(), pop(), pop(), pop(), pop());
+		e = pop();
+		d = pop();
+		c = pop();
+		b = pop();
+		a = pop();
+		warning("o8_roomOps: SO_ROOM_SATURATION(%d, %d, %d, %d, %d)", a, b, c, d, e);
+		// FIXME - this probably has the same format as for darkenPalette:
+		// thre values for R, G, B and a start/end palette range to modify.
+		// Now, how on earth does on modify the saturation of a single color channel?
+		// Change the hue/saturation of a color, no problem, I know how to do that,
+		// but for only a channel alone, I don't even know what that should mean... :-/
 		break;
 	default:
 		error("o8_roomOps: default case %d", subOp);