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

[lordhoto at users.sourceforge.net]

Revision: 44323
          http://scummvm.svn.sourceforge.net/scummvm/?rev=44323&view=rev
Author:   lordhoto
Date:     2009-09-25 01:00:34 +0000 (Fri, 25 Sep 2009)

Log Message:
-----------
Add comment about why r44322 was needed.

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

Modified: scummvm/trunk/engines/scumm/gfx.cpp
===================================================================
--- scummvm/trunk/engines/scumm/gfx.cpp	2009-09-25 00:50:51 UTC (rev 44322)
+++ scummvm/trunk/engines/scumm/gfx.cpp	2009-09-25 01:00:34 UTC (rev 44323)
@@ -3189,6 +3189,10 @@
 }
 
 void Gdi::writeRoomColor(byte *dst, byte color) const {
+	// As described in bug #1294513 "FOA/Amiga: Palette problem (Regression)"
+	// the original AMIGA version of Indy4: The Fate of Atlantis allowed
+	// overflowing of the palette index. To have the same result in our code,
+	// we need to do an logical AND 0xFF here to keep the result in [0, 255].
 	*dst = _roomPalette[(color + _paletteMod) & 0xFF];
 }
 


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