BUGFIX: MouseSafety does it's own parameter validation, and this calculation was...
[reactos.git] / reactos / subsys / win32k / eng / bitblt.c
index 38dd7db..6293a3e 100644 (file)
@@ -16,7 +16,7 @@
  *  along with this program; if not, write to the Free Software
  *  Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
  */
-/* $Id: bitblt.c,v 1.39 2004/01/16 19:32:00 gvg Exp $
+/* $Id: bitblt.c,v 1.60 2004/12/12 21:38:25 royce Exp $
  *
  * COPYRIGHT:        See COPYING in the top level directory
  * PROJECT:          ReactOS kernel
  * REVISION HISTORY:
  *        2/10/1999: Created
  */
-
-#include <ddk/winddi.h>
-#include <ddk/ntddk.h>
-#include <ddk/ntddmou.h>
-#include <ntos/minmax.h>
-#include "brush.h"
-#include "clip.h"
-#include "objects.h"
-#include "../dib/dib.h"
-#include "misc.h"
-#include <include/mouse.h>
-#include <include/object.h>
-#include <include/dib.h>
-#include <include/surface.h>
-#include <include/inteng.h>
-
-//#define NDEBUG
-#include <win32k/debug1.h>
+#include <w32k.h>
 
 typedef BOOLEAN STDCALL (*PBLTRECTFUNC)(SURFOBJ* OutputObj,
-                                        SURFGDI* OutputGDI,
                                         SURFOBJ* InputObj,
-                                        SURFGDI* InputGDI,
                                         SURFOBJ* Mask,
                                         XLATEOBJ* ColorTranslation,
                                         RECTL* OutputRect,
@@ -58,10 +39,9 @@ typedef BOOLEAN STDCALL (*PBLTRECTFUNC)(SURFOBJ* OutputObj,
                                         POINTL* BrushOrigin,
                                         ROP4 Rop4);
 typedef BOOLEAN STDCALL (*PSTRETCHRECTFUNC)(SURFOBJ* OutputObj,
-                                            SURFGDI* OutputGDI,
                                             SURFOBJ* InputObj,
-                                            SURFGDI* InputGDI,
                                             SURFOBJ* Mask,
+                                            CLIPOBJ* ClipRegion,
                                             XLATEOBJ* ColorTranslation,
                                             RECTL* OutputRect,
                                             RECTL* InputRect,
@@ -94,9 +74,7 @@ BOOL STDCALL EngIntersectRect(RECTL* prcDst, RECTL* prcSrc1, RECTL* prcSrc2)
 
 static BOOLEAN STDCALL
 BltMask(SURFOBJ* Dest,
-       SURFGDI* DestGDI,
        SURFOBJ* Source,
-       SURFGDI* SourceGDI,
        SURFOBJ* Mask, 
        XLATEOBJ* ColorTranslation,
        RECTL* DestRect,
@@ -106,48 +84,87 @@ BltMask(SURFOBJ* Dest,
        POINTL* BrushPoint,
        ROP4 Rop4)
 {
-  LONG i, j, dx, dy, c8;
-  BYTE *tMask, *lMask;
-  static BYTE maskbit[8] = { 0x80, 0x40, 0x20, 0x10, 0x08, 0x04, 0x02, 0x01 };
+   LONG i, j, dx, dy, c8;
+   BYTE *tMask, *lMask;
+   static BYTE maskbit[8] = { 0x80, 0x40, 0x20, 0x10, 0x08, 0x04, 0x02, 0x01 };
+   /* Pattern brushes */
+   PGDIBRUSHINST GdiBrush = NULL;
+   HBITMAP PatternSurface;
+   SURFOBJ *PatternObj = NULL;
+   PBITMAPOBJ PatternBitmap;
+   ULONG PatternWidth = 0, PatternHeight = 0, PatternY = 0;
   
-  dx = DestRect->right  - DestRect->left;
-  dy = DestRect->bottom - DestRect->top;
-
-  if (Mask != NULL)
-    {
-      tMask = Mask->pvBits + SourcePoint->y * Mask->lDelta + (SourcePoint->x >> 3);
-      for (j = 0; j < dy; j++)
-       {
-         lMask = tMask;
-         c8 = SourcePoint->x & 0x07;
-         for (i = 0; i < dx; i++)
-           {
-             if (0 != (*lMask & maskbit[c8]))
-               {
-                 DestGDI->DIB_PutPixel(Dest, DestRect->left + i, DestRect->top + j, Brush->iSolidColor);
-               }
-             c8++;
-             if (8 == c8)
-               {
-                 lMask++;
-                 c8=0;
-               }
-           }
-         tMask += Mask->lDelta;
-       }
-      return TRUE;
-    }
-  else
-    {
-    return FALSE;
-    }
+   if (Mask == NULL)
+   {
+      return FALSE;
+   }
+
+   dx = DestRect->right  - DestRect->left;
+   dy = DestRect->bottom - DestRect->top;
+
+   if (Brush->iSolidColor == 0xFFFFFFFF)
+   {
+      GdiBrush = CONTAINING_RECORD(
+         Brush,
+         GDIBRUSHINST,
+         BrushObject);
+
+      PatternSurface = GdiBrush->GdiBrushObject->hbmPattern;
+      PatternBitmap = BITMAPOBJ_LockBitmap(GdiBrush->GdiBrushObject->hbmPattern);
+      if(PatternBitmap != NULL)
+      {
+        PatternObj = &PatternBitmap->SurfObj;
+        PatternWidth = PatternObj->sizlBitmap.cx;
+        PatternHeight = PatternObj->sizlBitmap.cy;
+      }
+   }
+   else
+     PatternBitmap = NULL;
+
+   tMask = Mask->pvScan0 + SourcePoint->y * Mask->lDelta + (SourcePoint->x >> 3);
+   for (j = 0; j < dy; j++)
+   {
+      lMask = tMask;
+      c8 = SourcePoint->x & 0x07;
+      
+      if(PatternBitmap != NULL)
+         PatternY = (DestRect->top + j) % PatternHeight;
+      
+      for (i = 0; i < dx; i++)
+      {
+         if (0 != (*lMask & maskbit[c8]))
+         {
+            if (PatternBitmap == NULL)
+            {
+               DibFunctionsForBitmapFormat[Dest->iBitmapFormat].DIB_PutPixel(
+                  Dest, DestRect->left + i, DestRect->top + j, Brush->iSolidColor);
+            }
+            else
+            {
+               DibFunctionsForBitmapFormat[Dest->iBitmapFormat].DIB_PutPixel(
+                  Dest, DestRect->left + i, DestRect->top + j,
+                  DIB_GetSource(PatternObj, (DestRect->left + i) % PatternWidth, PatternY, GdiBrush->XlateObject));
+            }
+         }
+         c8++;
+         if (8 == c8)
+         {
+            lMask++;
+            c8 = 0;
+         }
+      }
+      tMask += Mask->lDelta;
+   }
+
+   if (PatternBitmap != NULL)
+      BITMAPOBJ_UnlockBitmap(PatternSurface);
+
+   return TRUE;
 }
 
 static BOOLEAN STDCALL
 BltPatCopy(SURFOBJ* Dest,
-          SURFGDI* DestGDI,
           SURFOBJ* Source,
-          SURFGDI* SourceGDI,
           SURFOBJ* Mask, 
           XLATEOBJ* ColorTranslation,
           RECTL* DestRect,
@@ -165,7 +182,8 @@ BltPatCopy(SURFOBJ* Dest,
   LineWidth  = DestRect->right - DestRect->left;
   for (y = DestRect->top; y < DestRect->bottom; y++)
   {
-    DestGDI->DIB_HLine(Dest, DestRect->left, DestRect->right, y,  Brush->iSolidColor);
+    DibFunctionsForBitmapFormat[Dest->iBitmapFormat].DIB_HLine(
+      Dest, DestRect->left, DestRect->right, y,  Brush->iSolidColor);
   }
 
   return TRUE;
@@ -173,9 +191,7 @@ BltPatCopy(SURFOBJ* Dest,
 
 static BOOLEAN STDCALL
 CallDibBitBlt(SURFOBJ* OutputObj,
-              SURFGDI* OutputGDI,
               SURFOBJ* InputObj,
-              SURFGDI* InputGDI,
               SURFOBJ* Mask,
               XLATEOBJ* ColorTranslation,
               RECTL* OutputRect,
@@ -185,7 +201,54 @@ CallDibBitBlt(SURFOBJ* OutputObj,
               POINTL* BrushOrigin,
               ROP4 Rop4)
 {
-  return OutputGDI->DIB_BitBlt(OutputObj, InputObj, OutputGDI, InputGDI, OutputRect, InputPoint, Brush, BrushOrigin, ColorTranslation, Rop4);
+   BLTINFO BltInfo;
+   PGDIBRUSHINST GdiBrush = NULL;
+   BITMAPOBJ *bmPattern;
+   BOOLEAN Result;
+
+   BltInfo.DestSurface = OutputObj;
+   BltInfo.SourceSurface = InputObj;
+   BltInfo.PatternSurface = NULL;
+   BltInfo.XlateSourceToDest = ColorTranslation;
+   BltInfo.DestRect = *OutputRect;
+   BltInfo.SourcePoint = *InputPoint;
+
+   if (Rop4 == SRCCOPY)
+      return DibFunctionsForBitmapFormat[OutputObj->iBitmapFormat].DIB_BitBltSrcCopy(&BltInfo);
+
+   BltInfo.XlatePatternToDest = NULL;
+   BltInfo.Brush = Brush;
+   BltInfo.BrushOrigin = *BrushOrigin;
+   BltInfo.Rop4 = Rop4;
+
+   /* Pattern brush */
+   if (ROP_USES_PATTERN(Rop4) && Brush->iSolidColor == 0xFFFFFFFF)
+   {
+      GdiBrush = CONTAINING_RECORD(Brush, GDIBRUSHINST, BrushObject);
+      if((bmPattern = BITMAPOBJ_LockBitmap(GdiBrush->GdiBrushObject->hbmPattern)))
+      {
+        BltInfo.PatternSurface = &bmPattern->SurfObj;
+      }
+      else
+      {
+        /* FIXME - What to do here? */
+      }
+      BltInfo.XlatePatternToDest = GdiBrush->XlateObject;
+   }
+   else
+   {
+     bmPattern = NULL;
+   }
+
+   Result = DibFunctionsForBitmapFormat[OutputObj->iBitmapFormat].DIB_BitBlt(&BltInfo);
+
+   /* Pattern brush */
+   if (bmPattern != NULL)
+   {
+      BITMAPOBJ_UnlockBitmap(BltInfo.PatternSurface->hsurf);
+   }
+
+   return Result;
 }
 
 INT abs(INT nm);
@@ -210,8 +273,6 @@ EngBitBlt(SURFOBJ *DestObj,
   RECTL              CombinedRect;
   RECT_ENUM          RectEnum;
   BOOL               EnumMore;
-  SURFGDI*           OutputGDI;
-  SURFGDI*           InputGDI;
   POINTL             InputPoint;
   RECTL              InputRect;
   RECTL              OutputRect;
@@ -221,17 +282,18 @@ EngBitBlt(SURFOBJ *DestObj,
   SURFOBJ*           InputObj;
   SURFOBJ*           OutputObj;
   PBLTRECTFUNC       BltRectFunc;
-  BOOLEAN            Ret;
+  BOOLEAN            Ret = TRUE;
   RECTL              ClipRect;
   unsigned           i;
   POINTL             Pt;
   ULONG              Direction;
   BOOL               UsesSource;
   BOOL               UsesPattern;
+  POINTL             AdjustedBrushOrigin;
 
   UsesSource = ((Rop4 & 0xCC0000) >> 2) != (Rop4 & 0x330000);
   UsesPattern = ((Rop4 & 0xF00000) >> 4) != (Rop4 & 0x0F0000);
-  if (! UsesSource && ! UsesPattern && 0xaacc != Rop4)
+  if (ROP_NOOP == Rop4)
     {
     /* Copy destination onto itself: nop */
     return TRUE;
@@ -268,15 +330,6 @@ EngBitBlt(SURFOBJ *DestObj,
     InputPoint.y = 0;
     }
 
-  if (NULL != InputObj)
-    {
-    InputGDI = (SURFGDI*) AccessInternalObjectFromUserObject(InputObj);
-    }
-  else
-    {
-      InputGDI = NULL;
-    }
-
   OutputRect = *DestRect;
   if (NULL != ClipRegion)
     {
@@ -322,11 +375,14 @@ EngBitBlt(SURFOBJ *DestObj,
   OutputRect.right = DestRect->right + Translate.x;
   OutputRect.top = DestRect->top + Translate.y;
   OutputRect.bottom = DestRect->bottom + Translate.y;
-
-  if (NULL != OutputObj)
-    {
-    OutputGDI = (SURFGDI*)AccessInternalObjectFromUserObject(OutputObj);
-    }
+  
+  if(BrushOrigin)
+  {
+    AdjustedBrushOrigin.x = BrushOrigin->x + Translate.x;
+    AdjustedBrushOrigin.y = BrushOrigin->y + Translate.y;
+  }
+  else
+    AdjustedBrushOrigin = Translate;
 
   // Determine clipping type
   if (ClipRegion == (CLIPOBJ *) NULL)
@@ -342,7 +398,14 @@ EngBitBlt(SURFOBJ *DestObj,
     }
   else if (PATCOPY == Rop4)
     {
+#if 0
       BltRectFunc = BltPatCopy;
+#else
+      if (Brush->iSolidColor == 0xFFFFFFFF)
+        BltRectFunc = CallDibBitBlt;
+      else
+        BltRectFunc = BltPatCopy;
+#endif
     }
   else
     {
@@ -353,8 +416,8 @@ EngBitBlt(SURFOBJ *DestObj,
   switch(clippingType)
   {
     case DC_TRIVIAL:
-      Ret = (*BltRectFunc)(OutputObj, OutputGDI, InputObj, InputGDI, Mask, ColorTranslation,
-                           &OutputRect, &InputPoint, MaskOrigin, Brush, BrushOrigin, Rop4);
+      Ret = (*BltRectFunc)(OutputObj, InputObj, Mask, ColorTranslation,
+                           &OutputRect, &InputPoint, MaskOrigin, Brush, &AdjustedBrushOrigin, Rop4);
       break;
     case DC_RECT:
       // Clip the blt to the clip rectangle
@@ -365,8 +428,8 @@ EngBitBlt(SURFOBJ *DestObj,
       EngIntersectRect(&CombinedRect, &OutputRect, &ClipRect);
       Pt.x = InputPoint.x + CombinedRect.left - OutputRect.left;
       Pt.y = InputPoint.y + CombinedRect.top - OutputRect.top;
-      Ret = (*BltRectFunc)(OutputObj, OutputGDI, InputObj, InputGDI, Mask, ColorTranslation,
-                           &CombinedRect, &Pt, MaskOrigin, Brush, BrushOrigin, Rop4);
+      Ret = (*BltRectFunc)(OutputObj, InputObj, Mask, ColorTranslation,
+                           &CombinedRect, &Pt, MaskOrigin, Brush, &AdjustedBrushOrigin, Rop4);
       break;
     case DC_COMPLEX:
       Ret = TRUE;
@@ -399,8 +462,8 @@ EngBitBlt(SURFOBJ *DestObj,
              EngIntersectRect(&CombinedRect, &OutputRect, &ClipRect);
              Pt.x = InputPoint.x + CombinedRect.left - OutputRect.left;
              Pt.y = InputPoint.y + CombinedRect.top - OutputRect.top;
-             Ret = (*BltRectFunc)(OutputObj, OutputGDI, InputObj, InputGDI, Mask, ColorTranslation,
-                                  &CombinedRect, &Pt, MaskOrigin, Brush, BrushOrigin, Rop4) &&
+             Ret = (*BltRectFunc)(OutputObj, InputObj, Mask, ColorTranslation,
+                                  &CombinedRect, &Pt, MaskOrigin, Brush, &AdjustedBrushOrigin, Rop4) &&
                    Ret;
            }
        }
@@ -416,9 +479,9 @@ EngBitBlt(SURFOBJ *DestObj,
 }
 
 BOOL STDCALL
-IntEngBitBlt(SURFOBJ *DestObj,
-             SURFOBJ *SourceObj,
-             SURFOBJ *Mask,
+IntEngBitBlt(BITMAPOBJ *DestObj,
+             BITMAPOBJ *SourceObj,
+             BITMAPOBJ *MaskObj,
              CLIPOBJ *ClipRegion,
              XLATEOBJ *ColorTranslation,
              RECTL *DestRect,
@@ -429,21 +492,67 @@ IntEngBitBlt(SURFOBJ *DestObj,
              ROP4 Rop4)
 {
   BOOLEAN ret;
-  SURFGDI *DestGDI;
-  SURFGDI *SourceGDI;
+  RECTL InputClippedRect;
   RECTL OutputRect;
   POINTL InputPoint;
+  BOOLEAN UsesSource;
+  SURFOBJ *DestSurf = &DestObj->SurfObj;
+  SURFOBJ *SourceSurf = SourceObj ? &SourceObj->SurfObj : NULL;
+  SURFOBJ *MaskSurf = MaskObj ? &MaskObj->SurfObj : NULL;
 
-  if (NULL != SourcePoint)
+  InputClippedRect = *DestRect;
+  if (InputClippedRect.right < InputClippedRect.left)
     {
+      InputClippedRect.left = DestRect->right;
+      InputClippedRect.right = DestRect->left;
+    }
+  if (InputClippedRect.bottom < InputClippedRect.top)
+    {
+      InputClippedRect.top = DestRect->bottom;
+      InputClippedRect.bottom = DestRect->top;
+    }
+  UsesSource = ((Rop4 & 0xCC0000) >> 2) != (Rop4 & 0x330000);
+  if (UsesSource)
+    {
+      if (NULL == SourcePoint || NULL == SourceObj)
+        {
+          return FALSE;
+        }
       InputPoint = *SourcePoint;
+
+      /* Make sure we don't try to copy anything outside the valid source region */
+      if (InputPoint.x < 0)
+        {
+          InputClippedRect.left -= InputPoint.x;
+          InputPoint.x = 0;
+        }
+      if (InputPoint.y < 0)
+        {
+          InputClippedRect.top -= InputPoint.y;
+          InputPoint.y = 0;
+        }
+      if (SourceSurf->sizlBitmap.cx < InputPoint.x + InputClippedRect.right - InputClippedRect.left)
+        {
+          InputClippedRect.right = InputClippedRect.left + SourceSurf->sizlBitmap.cx - InputPoint.x;
+        }
+      if (SourceSurf->sizlBitmap.cy < InputPoint.y + InputClippedRect.bottom - InputClippedRect.top)
+        {
+          InputClippedRect.bottom = InputClippedRect.top + SourceSurf->sizlBitmap.cy - InputPoint.y;
+        }
+
+      if (InputClippedRect.right < InputClippedRect.left ||
+          InputClippedRect.bottom < InputClippedRect.top)
+        {
+          /* Everything clipped away, nothing to do */
+          return TRUE;
+        }
     }
 
   /* Clip against the bounds of the clipping region so we won't try to write
    * outside the surface */
   if (NULL != ClipRegion)
     {
-      if (! EngIntersectRect(&OutputRect, DestRect, &ClipRegion->rclBounds))
+      if (! EngIntersectRect(&OutputRect, &InputClippedRect, &ClipRegion->rclBounds))
        {
          return TRUE;
        }
@@ -452,44 +561,41 @@ IntEngBitBlt(SURFOBJ *DestObj,
     }
   else
     {
-      OutputRect = *DestRect;
+      OutputRect = InputClippedRect;
     }
 
-  if (NULL != SourceObj)
+  if (UsesSource)
     {
-    SourceGDI = (SURFGDI*) AccessInternalObjectFromUserObject(SourceObj);
-    MouseSafetyOnDrawStart(SourceObj, SourceGDI, InputPoint.x, InputPoint.y,
+    MouseSafetyOnDrawStart(SourceSurf, InputPoint.x, InputPoint.y,
                            (InputPoint.x + abs(DestRect->right - DestRect->left)),
                           (InputPoint.y + abs(DestRect->bottom - DestRect->top)));
     }
 
   /* No success yet */
   ret = FALSE;
-  DestGDI = (SURFGDI*)AccessInternalObjectFromUserObject(DestObj);
-  MouseSafetyOnDrawStart(DestObj, DestGDI, OutputRect.left, OutputRect.top,
+  MouseSafetyOnDrawStart(DestSurf, OutputRect.left, OutputRect.top,
                          OutputRect.right, OutputRect.bottom);
 
   /* Call the driver's DrvBitBlt if available */
-  if (NULL != DestGDI->BitBlt)
+  if (DestObj->flHooks & HOOK_BITBLT)
     {
-      ExAcquireFastMutex(DestGDI->DriverLock);
-      ret = DestGDI->BitBlt(DestObj, SourceObj, Mask, ClipRegion, ColorTranslation,
+      ret = GDIDEVFUNCS(DestSurf).BitBlt(
+                            DestSurf, SourceSurf, MaskSurf, ClipRegion, ColorTranslation,
                             &OutputRect, &InputPoint, MaskOrigin, Brush, BrushOrigin,
                             Rop4);
-      ExReleaseFastMutex(DestGDI->DriverLock);
     }
 
   if (! ret)
     {
-      ret = EngBitBlt(DestObj, SourceObj, Mask, ClipRegion, ColorTranslation,
+      ret = EngBitBlt(DestSurf, SourceSurf, MaskSurf, ClipRegion, ColorTranslation,
                       &OutputRect, &InputPoint, MaskOrigin, Brush, BrushOrigin,
                       Rop4);
     }
 
-  MouseSafetyOnDrawEnd(DestObj, DestGDI);
-  if (NULL != SourceObj)
+  MouseSafetyOnDrawEnd(DestSurf);
+  if (UsesSource)
     {
-    MouseSafetyOnDrawEnd(SourceObj, SourceGDI);
+    MouseSafetyOnDrawEnd(SourceSurf);
     }
 
   return ret;
@@ -497,10 +603,9 @@ IntEngBitBlt(SURFOBJ *DestObj,
 
 static BOOLEAN STDCALL
 CallDibStretchBlt(SURFOBJ* OutputObj,
-                  SURFGDI* OutputGDI,
                   SURFOBJ* InputObj,
-                  SURFGDI* InputGDI,
                   SURFOBJ* Mask,
+                 CLIPOBJ* ClipRegion,
                   XLATEOBJ* ColorTranslation,
                   RECTL* OutputRect,
                   RECTL* InputRect,
@@ -508,7 +613,17 @@ CallDibStretchBlt(SURFOBJ* OutputObj,
                   POINTL* BrushOrigin,
                   ULONG Mode)
 {
-  return OutputGDI->DIB_StretchBlt(OutputObj, InputObj, OutputGDI, InputGDI, OutputRect, InputRect, MaskOrigin, BrushOrigin, ColorTranslation, Mode);
+  POINTL RealBrushOrigin;
+  if (BrushOrigin == NULL)
+    {
+      RealBrushOrigin.x = RealBrushOrigin.y = 0;
+    }
+  else
+    {
+      RealBrushOrigin = *BrushOrigin;
+    }
+  return DibFunctionsForBitmapFormat[OutputObj->iBitmapFormat].DIB_StretchBlt(
+    OutputObj, InputObj, OutputRect, InputRect, MaskOrigin, RealBrushOrigin, ClipRegion, ColorTranslation, Mode);
 }
 
 
@@ -530,12 +645,6 @@ EngStretchBlt(
 {
   // www.osr.com/ddk/graphics/gdifncs_0bs7.htm
   
-  BYTE               clippingType;
-  RECTL              CombinedRect;
-//  RECT_ENUM          RectEnum;
-//  BOOL               EnumMore;
-  SURFGDI*           OutputGDI;
-  SURFGDI*           InputGDI;
   POINTL             InputPoint;
   RECTL              InputRect;
   RECTL              OutputRect;
@@ -544,95 +653,55 @@ EngStretchBlt(
   INTENG_ENTER_LEAVE EnterLeaveDest;
   SURFOBJ*           InputObj;
   SURFOBJ*           OutputObj;
-  PSTRETCHRECTFUNC       BltRectFunc;
+  PSTRETCHRECTFUNC   BltRectFunc;
   BOOLEAN            Ret;
-  RECTL              ClipRect;
-//  unsigned           i;
-  POINTL             Pt;
-//  ULONG              Direction;
+  POINTL             AdjustedBrushOrigin;
 
-    InputRect.left = prclSrc->left;
-    InputRect.right = prclSrc->right;
-    InputRect.top = prclSrc->top;
-    InputRect.bottom = prclSrc->bottom;
+  InputRect.left = prclSrc->left;
+  InputRect.right = prclSrc->right;
+  InputRect.top = prclSrc->top;
+  InputRect.bottom = prclSrc->bottom;
 
   if (! IntEngEnter(&EnterLeaveSource, SourceObj, &InputRect, TRUE, &Translate, &InputObj))
     {
-    return FALSE;
+      return FALSE;
     }
 
-   InputPoint.x = InputRect.left + Translate.x;
-   InputPoint.y = InputRect.top + Translate.y;
+  InputPoint.x = InputRect.left + Translate.x;
+  InputPoint.y = InputRect.top + Translate.y;
  
-  if (NULL != InputObj)
-    {
-    InputGDI = (SURFGDI*) AccessInternalObjectFromUserObject(InputObj);
-    }
-  else
-    {
-      InputGDI = NULL;
-    }
-
   OutputRect = *prclDest;
-  if (NULL != ClipRegion)
-    {
-      if (OutputRect.left < ClipRegion->rclBounds.left)
-       {
-         InputRect.left += ClipRegion->rclBounds.left - OutputRect.left;
-         InputPoint.x += ClipRegion->rclBounds.left - OutputRect.left;
-         OutputRect.left = ClipRegion->rclBounds.left;
-       }
-      if (ClipRegion->rclBounds.right < OutputRect.right)
-       {
-         InputRect.right -=  OutputRect.right - ClipRegion->rclBounds.right;
-         OutputRect.right = ClipRegion->rclBounds.right;
-       }
-      if (OutputRect.top < ClipRegion->rclBounds.top)
-       {
-         InputRect.top += ClipRegion->rclBounds.top - OutputRect.top;
-         InputPoint.y += ClipRegion->rclBounds.top - OutputRect.top;
-         OutputRect.top = ClipRegion->rclBounds.top;
-       }
-      if (ClipRegion->rclBounds.bottom < OutputRect.bottom)
-       {
-         InputRect.bottom -=  OutputRect.bottom - ClipRegion->rclBounds.bottom;
-         OutputRect.bottom = ClipRegion->rclBounds.bottom;
-       }
-    }
 
   /* Check for degenerate case: if height or width of OutputRect is 0 pixels there's
      nothing to do */
   if (OutputRect.right <= OutputRect.left || OutputRect.bottom <= OutputRect.top)
     {
-    IntEngLeave(&EnterLeaveSource);
-    return TRUE;
+      IntEngLeave(&EnterLeaveSource);
+      return TRUE;
     }
 
   if (! IntEngEnter(&EnterLeaveDest, DestObj, &OutputRect, FALSE, &Translate, &OutputObj))
     {
-    IntEngLeave(&EnterLeaveSource);
-    return FALSE;
+      IntEngLeave(&EnterLeaveSource);
+      return FALSE;
     }
 
   OutputRect.left = prclDest->left + Translate.x;
   OutputRect.right = prclDest->right + Translate.x;
   OutputRect.top = prclDest->top + Translate.y;
   OutputRect.bottom = prclDest->bottom + Translate.y;
-
-  if (NULL != OutputObj)
+  
+  if (NULL != BrushOrigin)
     {
-    OutputGDI = (SURFGDI*)AccessInternalObjectFromUserObject(OutputObj);
+      AdjustedBrushOrigin.x = BrushOrigin->x + Translate.x;
+      AdjustedBrushOrigin.y = BrushOrigin->y + Translate.y;
+    }
+  else
+    {
+      AdjustedBrushOrigin = Translate;
     }
 
-  // Determine clipping type
-  if (ClipRegion == (CLIPOBJ *) NULL)
-  {
-    clippingType = DC_TRIVIAL;
-  } else {
-    clippingType = ClipRegion->iDComplexity;
-  }
-
-  if (Mask != NULL)//(0xaacc == Rop4)
+  if (Mask != NULL)
     {
       //BltRectFunc = BltMask;
       DPRINT("EngStretchBlt isn't capable of handling mask yet.\n");
@@ -647,70 +716,9 @@ EngStretchBlt(
     }
 
 
-  switch(clippingType)
-  {
-    case DC_TRIVIAL:
-      Ret = (*BltRectFunc)(OutputObj, OutputGDI, InputObj, InputGDI, Mask, ColorTranslation,
-                          &OutputRect, &InputRect, MaskOrigin, BrushOrigin, Mode);
-      break;
-    case DC_RECT:
-      // Clip the blt to the clip rectangle
-      ClipRect.left = ClipRegion->rclBounds.left + Translate.x;
-      ClipRect.right = ClipRegion->rclBounds.right + Translate.x;
-      ClipRect.top = ClipRegion->rclBounds.top + Translate.y;
-      ClipRect.bottom = ClipRegion->rclBounds.bottom + Translate.y;
-      EngIntersectRect(&CombinedRect, &OutputRect, &ClipRect);
-      Pt.x = InputPoint.x + CombinedRect.left - OutputRect.left;
-      Pt.y = InputPoint.y + CombinedRect.top - OutputRect.top;
-      Ret = (*BltRectFunc)(OutputObj, OutputGDI, InputObj, InputGDI, Mask, ColorTranslation,
-                           &OutputRect, &InputRect, MaskOrigin, BrushOrigin, Mode);
-      //Ret = (*BltRectFunc)(OutputObj, OutputGDI, InputObj, InputGDI, Mask, ColorTranslation,
-      //                     &CombinedRect, &Pt, MaskOrigin, Brush, BrushOrigin, Rop4);
-      DPRINT("EngStretchBlt() doesn't support DC_RECT clipping yet, so blitting w/o clip.\n");
-      break;
-      // TODO: Complex clipping
-    /*
-    case DC_COMPLEX:
-      Ret = TRUE;
-      if (OutputObj == InputObj)
-       {
-         if (OutputRect.top < InputPoint.y)
-           {
-             Direction = OutputRect.left < InputPoint.x ? CD_RIGHTDOWN : CD_LEFTDOWN;
-           }
-         else
-           {
-             Direction = OutputRect.left < InputPoint.x ? CD_RIGHTUP : CD_LEFTUP;
-           }
-       }
-      else
-       {
-         Direction = CD_ANY;
-       }
-      CLIPOBJ_cEnumStart(ClipRegion, FALSE, CT_RECTANGLES, Direction, 0);
-      do
-       {
-         EnumMore = CLIPOBJ_bEnum(ClipRegion,(ULONG) sizeof(RectEnum), (PVOID) &RectEnum);
-
-         for (i = 0; i < RectEnum.c; i++)
-           {
-             ClipRect.left = RectEnum.arcl[i].left + Translate.x;
-             ClipRect.right = RectEnum.arcl[i].right + Translate.x;
-             ClipRect.top = RectEnum.arcl[i].top + Translate.y;
-             ClipRect.bottom = RectEnum.arcl[i].bottom + Translate.y;
-             EngIntersectRect(&CombinedRect, &OutputRect, &ClipRect);
-             Pt.x = InputPoint.x + CombinedRect.left - OutputRect.left;
-             Pt.y = InputPoint.y + CombinedRect.top - OutputRect.top;
-             Ret = (*BltRectFunc)(OutputObj, OutputGDI, InputObj, InputGDI, Mask, ColorTranslation,
-                                  &CombinedRect, &Pt, MaskOrigin, Brush, BrushOrigin, Rop4) &&
-                   Ret;
-           }
-       }
-      while(EnumMore);
-      break;
-      */
-  }
-
+  Ret = (*BltRectFunc)(OutputObj, InputObj, Mask, ClipRegion,
+                       ColorTranslation, &OutputRect, &InputRect, MaskOrigin,
+                       &AdjustedBrushOrigin, Mode);
 
   IntEngLeave(&EnterLeaveDest);
   IntEngLeave(&EnterLeaveSource);
@@ -719,94 +727,65 @@ EngStretchBlt(
 }
 
 BOOL STDCALL
-IntEngStretchBlt(SURFOBJ *DestObj,
-             SURFOBJ *SourceObj,
-             SURFOBJ *Mask,
-             CLIPOBJ *ClipRegion,
-             XLATEOBJ *ColorTranslation,
-             RECTL *DestRect,
-             RECTL *SourceRect,
-             POINTL *pMaskOrigin,
-             BRUSHOBJ *Brush,
-             POINTL *BrushOrigin,
-             ULONG Mode)
+IntEngStretchBlt(BITMAPOBJ *DestObj,
+                 BITMAPOBJ *SourceObj,
+                 BITMAPOBJ *MaskObj,
+                 CLIPOBJ *ClipRegion,
+                 XLATEOBJ *ColorTranslation,
+                 RECTL *DestRect,
+                 RECTL *SourceRect,
+                 POINTL *pMaskOrigin,
+                 BRUSHOBJ *Brush,
+                 POINTL *BrushOrigin,
+                 ULONG Mode)
 {
   BOOLEAN ret;
-  SURFGDI *DestGDI;
-  SURFGDI *SourceGDI;
-  RECTL OutputRect;
-  RECTL InputRect;
   COLORADJUSTMENT ca;
   POINT MaskOrigin;
+  SURFOBJ *DestSurf = &DestObj->SurfObj;
+  SURFOBJ *SourceSurf = SourceObj ? &SourceObj->SurfObj : NULL;
+  SURFOBJ *MaskSurf = MaskObj ? &MaskObj->SurfObj : NULL;
 
   if (pMaskOrigin != NULL)
     {
       MaskOrigin.x = pMaskOrigin->x; MaskOrigin.y = pMaskOrigin->y;
     }
 
-  if (NULL != SourceRect)
-    {
-      InputRect = *SourceRect;
-    }
-
-  // FIXME: Clipping is taken from IntEngBitBlt w/o modifications!
-  
-  /* Clip against the bounds of the clipping region so we won't try to write
-   * outside the surface */
-  if (NULL != ClipRegion)
-    {
-      if (! EngIntersectRect(&OutputRect, DestRect, &ClipRegion->rclBounds))
-       {
-         return TRUE;
-       }
-         DPRINT("Clipping isn't handled in IntEngStretchBlt() correctly yet\n");
-      //InputPoint.x += OutputRect.left - DestRect->left;
-      //InputPoint.y += OutputRect.top - DestRect->top;
-    }
-  else
-    {
-      OutputRect = *DestRect;
-    }
-
   if (NULL != SourceObj)
     {
-    SourceGDI = (SURFGDI*) AccessInternalObjectFromUserObject(SourceObj);
-    MouseSafetyOnDrawStart(SourceObj, SourceGDI, InputRect.left, InputRect.top,
-                           (InputRect.left + abs(InputRect.right - InputRect.left)),
-                          (InputRect.top + abs(InputRect.bottom - InputRect.top)));
+    MouseSafetyOnDrawStart(SourceSurf, SourceRect->left, SourceRect->top,
+                           SourceRect->right, SourceRect->bottom);
     }
 
   /* No success yet */
   ret = FALSE;
-  DestGDI = (SURFGDI*)AccessInternalObjectFromUserObject(DestObj);
-  MouseSafetyOnDrawStart(DestObj, DestGDI, OutputRect.left, OutputRect.top,
-                         OutputRect.right, OutputRect.bottom);
+  MouseSafetyOnDrawStart(DestSurf, DestRect->left, DestRect->top,
+                         DestRect->right, DestRect->bottom);
 
   /* Prepare color adjustment */
 
   /* Call the driver's DrvStretchBlt if available */
-  if (NULL != DestGDI->StretchBlt)
+  if (DestObj->flHooks & HOOK_STRETCHBLT)
     {
       /* Drv->StretchBlt (look at http://www.osr.com/ddk/graphics/ddifncs_3ew7.htm )
       SURFOBJ *psoMask // optional, if it exists, then rop4=0xCCAA, otherwise rop4=0xCCCC */
       // FIXME: MaskOrigin is always NULL !
-      ExAcquireFastMutex(DestGDI->DriverLock);
-      ret = DestGDI->StretchBlt(DestObj, SourceObj, Mask, ClipRegion, ColorTranslation,
-                            &ca, BrushOrigin, &OutputRect, &InputRect, NULL, Mode);
-      ExReleaseFastMutex(DestGDI->DriverLock);
+      ret = GDIDEVFUNCS(DestSurf).StretchBlt(
+                            DestSurf, SourceSurf, MaskSurf, ClipRegion, ColorTranslation,
+                            &ca, BrushOrigin, DestRect, SourceRect, NULL, Mode);
     }
 
   if (! ret)
     {
       // FIXME: see previous fixme
-      ret = EngStretchBlt(DestObj, SourceObj, Mask, ClipRegion, ColorTranslation,
-                          &ca, BrushOrigin, &OutputRect, &InputRect, NULL, Mode);
+      ret = EngStretchBlt(DestSurf, SourceSurf, MaskSurf, ClipRegion, ColorTranslation,
+                          &ca, BrushOrigin, DestRect, SourceRect, NULL, Mode);
     }
 
-  MouseSafetyOnDrawEnd(DestObj, DestGDI);
-  if (NULL != SourceObj)
+  MouseSafetyOnDrawEnd(DestSurf);
+  if (NULL != SourceSurf)
     {
-    MouseSafetyOnDrawEnd(SourceObj, SourceGDI);
+    MouseSafetyOnDrawEnd(SourceSurf);
     }
 
   return ret;
@@ -817,9 +796,7 @@ IntEngStretchBlt(SURFOBJ *DestObj,
 /* renders the alpha mask bitmap */
 static BOOLEAN STDCALL
 AlphaBltMask(SURFOBJ* Dest,
-       SURFGDI* DestGDI,
        SURFOBJ* Source,
-       SURFGDI* SourceGDI,
        SURFOBJ* Mask, 
        XLATEOBJ* ColorTranslation,
        XLATEOBJ* SrcColorTranslation,
@@ -844,7 +821,7 @@ AlphaBltMask(SURFOBJ* Dest,
       g = (int)GetGValue(BrushColor);
       b = (int)GetBValue(BrushColor);
       
-      tMask = Mask->pvBits + MaskPoint->y * Mask->lDelta + MaskPoint->x;
+      tMask = Mask->pvBits + (SourcePoint->y * Mask->lDelta) + SourcePoint->x;
       for (j = 0; j < dy; j++)
        {
          lMask = tMask;
@@ -854,11 +831,12 @@ AlphaBltMask(SURFOBJ* Dest,
                {
                        if(*lMask == 0xff)
                        {
-                               DestGDI->DIB_PutPixel(Dest, DestRect->left + i, DestRect->top + j, Brush->iSolidColor);
+                               DibFunctionsForBitmapFormat[Dest->iBitmapFormat].DIB_PutPixel(
+                                       Dest, DestRect->left + i, DestRect->top + j, Brush->iSolidColor);
                        }
                        else
                        {
-                               Background = DIB_GetSource(Dest, DestGDI, DestRect->left + i, DestRect->top + j, SrcColorTranslation);
+                               Background = DIB_GetSource(Dest, DestRect->left + i, DestRect->top + j, SrcColorTranslation);
 
                                NewColor = 
                                     RGB((*lMask * (r - GetRValue(Background)) >> 8) + GetRValue(Background),
@@ -866,7 +844,8 @@ AlphaBltMask(SURFOBJ* Dest,
                                         (*lMask * (b - GetBValue(Background)) >> 8) + GetBValue(Background));
                                
                                Background = XLATEOBJ_iXlate(ColorTranslation, NewColor);
-                               DestGDI->DIB_PutPixel(Dest, DestRect->left + i, DestRect->top + j, Background);
+                               DibFunctionsForBitmapFormat[Dest->iBitmapFormat].DIB_PutPixel(
+                                       Dest, DestRect->left + i, DestRect->top + j, Background);
                        }
                }
                  lMask++;
@@ -897,8 +876,6 @@ EngMaskBitBlt(SURFOBJ *DestObj,
   RECTL              CombinedRect;
   RECT_ENUM          RectEnum;
   BOOL               EnumMore;
-  SURFGDI*           OutputGDI;
-  SURFGDI*           InputGDI;
   POINTL             InputPoint;
   RECTL              InputRect;
   RECTL              OutputRect;
@@ -907,12 +884,12 @@ EngMaskBitBlt(SURFOBJ *DestObj,
   INTENG_ENTER_LEAVE EnterLeaveDest;
   SURFOBJ*           InputObj;
   SURFOBJ*           OutputObj;
-  BOOLEAN            Ret;
+  BOOLEAN            Ret = TRUE;
   RECTL              ClipRect;
   unsigned           i;
   POINTL             Pt;
   ULONG              Direction;
-  SURFGDI*           DestGDI;
+  POINTL             AdjustedBrushOrigin;
 
   if (NULL != SourcePoint)
     {
@@ -929,11 +906,8 @@ EngMaskBitBlt(SURFOBJ *DestObj,
     InputRect.bottom = DestRect->bottom - DestRect->top;
     }
 
-  DestGDI = (SURFGDI*)AccessInternalObjectFromUserObject(DestObj);
-  ExAcquireFastMutex(DestGDI->DriverLock);
   if (! IntEngEnter(&EnterLeaveSource, NULL, &InputRect, TRUE, &Translate, &InputObj))
     {
-    ExReleaseFastMutex(DestGDI->DriverLock);
     return FALSE;
     }
 
@@ -948,15 +922,6 @@ EngMaskBitBlt(SURFOBJ *DestObj,
     InputPoint.y = 0;
     }
 
-  if (NULL != InputObj)
-    {
-    InputGDI = (SURFGDI*) AccessInternalObjectFromUserObject(InputObj);
-    }
-  else
-    {
-      InputGDI = NULL;
-    }
-
   OutputRect = *DestRect;
   if (NULL != ClipRegion)
     {
@@ -989,14 +954,12 @@ EngMaskBitBlt(SURFOBJ *DestObj,
   if (OutputRect.right <= OutputRect.left || OutputRect.bottom <= OutputRect.top)
     {
     IntEngLeave(&EnterLeaveSource);
-    ExReleaseFastMutex(DestGDI->DriverLock);
     return TRUE;
     }
 
   if (! IntEngEnter(&EnterLeaveDest, DestObj, &OutputRect, FALSE, &Translate, &OutputObj))
     {
     IntEngLeave(&EnterLeaveSource);
-    ExReleaseFastMutex(DestGDI->DriverLock);
     return FALSE;
     }
 
@@ -1004,11 +967,14 @@ EngMaskBitBlt(SURFOBJ *DestObj,
   OutputRect.right = DestRect->right + Translate.x;
   OutputRect.top = DestRect->top + Translate.y;
   OutputRect.bottom = DestRect->bottom + Translate.y;
-
-  if (NULL != OutputObj)
-    {
-    OutputGDI = (SURFGDI*)AccessInternalObjectFromUserObject(OutputObj);
-    }
+  
+  if(BrushOrigin)
+  {
+    AdjustedBrushOrigin.x = BrushOrigin->x + Translate.x;
+    AdjustedBrushOrigin.y = BrushOrigin->y + Translate.y;
+  }
+  else
+    AdjustedBrushOrigin = Translate;
 
   // Determine clipping type
   if (ClipRegion == (CLIPOBJ *) NULL)
@@ -1022,11 +988,11 @@ EngMaskBitBlt(SURFOBJ *DestObj,
   {
     case DC_TRIVIAL:
       if(Mask->iBitmapFormat == BMF_8BPP)
-        Ret = AlphaBltMask(OutputObj, OutputGDI, InputObj, InputGDI, Mask, DestColorTranslation, SourceColorTranslation,
-                           &OutputRect, &InputPoint, MaskOrigin, Brush, BrushOrigin);
+        Ret = AlphaBltMask(OutputObj, InputObj, Mask, DestColorTranslation, SourceColorTranslation,
+                           &OutputRect, &InputPoint, MaskOrigin, Brush, &AdjustedBrushOrigin);
       else
-        Ret = BltMask(OutputObj, OutputGDI, InputObj, InputGDI, Mask, DestColorTranslation,
-                           &OutputRect, &InputPoint, MaskOrigin, Brush, BrushOrigin, 0xAACC);
+        Ret = BltMask(OutputObj, InputObj, Mask, DestColorTranslation,
+                           &OutputRect, &InputPoint, MaskOrigin, Brush, &AdjustedBrushOrigin, 0xAACC);
       break;
     case DC_RECT:
       // Clip the blt to the clip rectangle
@@ -1038,11 +1004,11 @@ EngMaskBitBlt(SURFOBJ *DestObj,
       Pt.x = InputPoint.x + CombinedRect.left - OutputRect.left;
       Pt.y = InputPoint.y + CombinedRect.top - OutputRect.top;
       if(Mask->iBitmapFormat == BMF_8BPP)
-        Ret = AlphaBltMask(OutputObj, OutputGDI, InputObj, InputGDI, Mask, DestColorTranslation, SourceColorTranslation,
-                           &CombinedRect, &Pt, MaskOrigin, Brush, BrushOrigin);
+        Ret = AlphaBltMask(OutputObj, InputObj, Mask, DestColorTranslation, SourceColorTranslation,
+                           &CombinedRect, &Pt, MaskOrigin, Brush, &AdjustedBrushOrigin);
       else
-        Ret = BltMask(OutputObj, OutputGDI, InputObj, InputGDI, Mask, DestColorTranslation,
-                           &CombinedRect, &Pt, MaskOrigin, Brush, BrushOrigin, 0xAACC);
+        Ret = BltMask(OutputObj, InputObj, Mask, DestColorTranslation,
+                           &CombinedRect, &Pt, MaskOrigin, Brush, &AdjustedBrushOrigin, 0xAACC);
       break;
     case DC_COMPLEX:
       Ret = TRUE;
@@ -1076,11 +1042,11 @@ EngMaskBitBlt(SURFOBJ *DestObj,
              Pt.x = InputPoint.x + CombinedRect.left - OutputRect.left;
              Pt.y = InputPoint.y + CombinedRect.top - OutputRect.top;
              if(Mask->iBitmapFormat == BMF_8BPP)
-               Ret = AlphaBltMask(OutputObj, OutputGDI, InputObj, InputGDI, Mask, DestColorTranslation, SourceColorTranslation,
-                                  &CombinedRect, &Pt, MaskOrigin, Brush, BrushOrigin) && Ret;
+               Ret = AlphaBltMask(OutputObj, InputObj, Mask, DestColorTranslation, SourceColorTranslation,
+                                  &CombinedRect, &Pt, MaskOrigin, Brush, &AdjustedBrushOrigin) && Ret;
               else
-                Ret = BltMask(OutputObj, OutputGDI, InputObj, InputGDI, Mask, DestColorTranslation,
-                                   &CombinedRect, &Pt, MaskOrigin, Brush, BrushOrigin, 0xAACC) && Ret;
+                Ret = BltMask(OutputObj, InputObj, Mask, DestColorTranslation,
+                                   &CombinedRect, &Pt, MaskOrigin, Brush, &AdjustedBrushOrigin, 0xAACC) && Ret;
            }
        }
       while(EnumMore);
@@ -1091,12 +1057,11 @@ EngMaskBitBlt(SURFOBJ *DestObj,
   IntEngLeave(&EnterLeaveDest);
   IntEngLeave(&EnterLeaveSource);
 
-  ExReleaseFastMutex(DestGDI->DriverLock);
-
   /* Dummy BitBlt to let driver know that something has changed.
      0x00AA0029 is the Rop for D (no-op) */
-  IntEngBitBlt(DestObj, NULL, Mask, ClipRegion, DestColorTranslation,
-               DestRect, SourcePoint, MaskOrigin, Brush, BrushOrigin, 0x00AA0029);
+  /* FIXME: Remove the typecast! */
+  IntEngBitBlt((BITMAPOBJ*)DestObj, NULL, (BITMAPOBJ*)Mask, ClipRegion, DestColorTranslation,
+               DestRect, SourcePoint, MaskOrigin, Brush, BrushOrigin, ROP_NOOP);
 
   return Ret;
 }
@@ -1114,7 +1079,6 @@ IntEngMaskBlt(SURFOBJ *DestObj,
              POINTL *BrushOrigin)
 {
   BOOLEAN ret;
-  SURFGDI *DestGDI;
   RECTL OutputRect;
   POINTL InputPoint;
 
@@ -1141,14 +1105,13 @@ IntEngMaskBlt(SURFOBJ *DestObj,
 
   /* No success yet */
   ret = FALSE;
-  DestGDI = (SURFGDI*)AccessInternalObjectFromUserObject(DestObj);
-  MouseSafetyOnDrawStart(DestObj, DestGDI, OutputRect.left, OutputRect.top,
+  MouseSafetyOnDrawStart(DestObj, OutputRect.left, OutputRect.top,
                          OutputRect.right, OutputRect.bottom);
 
   ret = EngMaskBitBlt(DestObj, Mask, ClipRegion, DestColorTranslation, SourceColorTranslation,
                       &OutputRect, &InputPoint, MaskOrigin, Brush, BrushOrigin);
 
-  MouseSafetyOnDrawEnd(DestObj, DestGDI);
+  MouseSafetyOnDrawEnd(DestObj);
 
   return ret;
 }