[Scummvm-cvs-logs] scummvm master -> bf6c42d816bd8e6f87c4ecb7459bafcecaf0d010

bluegr md5 at scummvm.org
Sat Jan 14 15:38:15 CET 2012


This automated email contains information about 1 new commit which have been
pushed to the 'scummvm' repo located at https://github.com/scummvm/scummvm .

Summary:
bf6c42d816 SCI: Don't clip the plane rectangle in kernelDeletePlane()


Commit: bf6c42d816bd8e6f87c4ecb7459bafcecaf0d010
    https://github.com/scummvm/scummvm/commit/bf6c42d816bd8e6f87c4ecb7459bafcecaf0d010
Author: Filippos Karapetis (md5 at scummvm.org)
Date: 2012-01-14T06:37:07-08:00

Commit Message:
SCI: Don't clip the plane rectangle in kernelDeletePlane()

The actual cause for this clipping has been fixed

Changed paths:
    engines/sci/graphics/frameout.cpp



diff --git a/engines/sci/graphics/frameout.cpp b/engines/sci/graphics/frameout.cpp
index 1402dc9..e3fac80 100644
--- a/engines/sci/graphics/frameout.cpp
+++ b/engines/sci/graphics/frameout.cpp
@@ -195,10 +195,6 @@ void GfxFrameout::kernelDeletePlane(reg_t object) {
 			planeRect.left = (planeRect.left * screenRect.width()) / _scriptsRunningWidth;
 			planeRect.bottom = (planeRect.bottom * screenRect.height()) / _scriptsRunningHeight;
 			planeRect.right = (planeRect.right * screenRect.width()) / _scriptsRunningWidth;
-			planeRect.clip(screenRect); // we need to do this, at least in gk1 on cemetary we get bottom right -> 201, 321
-			// FIXME: The code above incorrectly added 1 pixel to the plane's
-			// bottom and right, so probably the plane clipping code is no
-			// longer necessary
 			// Blackout removed plane rect
 			_paint32->fillRect(planeRect, 0);
 			return;






More information about the Scummvm-git-logs mailing list