[WIN32K]

[reactos.git] / reactos / win32ss / gdi / eng / engwindow.c

diff --git a/reactos/win32ss/gdi/eng/engwindow.c b/reactos/win32ss/gdi/eng/engwindow.c
index 423f61f..c5b4d93 100644 (file)
--- a/reactos/win32ss/gdi/eng/engwindow.c
+++ b/reactos/win32ss/gdi/eng/engwindow.c
@@ -7,9 +7,8 @@
  */
 
 #include <win32k.h>
-
-#define NDEBUG
 #include <debug.h>
+DBG_DEFAULT_CHANNEL(EngWnd);
 
 INT gcountPWO = 0;
 
@@ -42,7 +41,7 @@ IntEngWndCallChangeProc(
         pwo = NULL;
     }
 
-    DPRINT("Calling WNDOBJCHANGEPROC (0x%p), Changed = 0x%x\n",
+    TRACE("Calling WNDOBJCHANGEPROC (0x%p), Changed = 0x%x\n",
            WndObjInt->ChangeProc, flChanged);
     WndObjInt->ChangeProc(pwo, flChanged);
 }
@@ -61,12 +60,11 @@ IntEngWndUpdateClipObj(
     CLIPOBJ *ClipObj = NULL;
     CLIPOBJ *OldClipObj;
 
-    DPRINT("IntEngWndUpdateClipObj\n");
+    TRACE("IntEngWndUpdateClipObj\n");
 
     hVisRgn = VIS_ComputeVisibleRegion(Window, TRUE, TRUE, TRUE);
     if (hVisRgn != NULL)
     {
-        NtGdiOffsetRgn(hVisRgn, Window->rcClient.left, Window->rcClient.top);
         visRgn = RGNOBJAPI_Lock(hVisRgn, NULL);
         if (visRgn != NULL)
         {
@@ -74,15 +72,15 @@ IntEngWndUpdateClipObj(
             {
                 ClipObj = IntEngCreateClipRegion(visRgn->rdh.nCount, visRgn->Buffer,
                                                  &visRgn->rdh.rcBound);
-                DPRINT("Created visible region with %lu rects\n", visRgn->rdh.nCount);
-                DPRINT("  BoundingRect: %d, %d  %d, %d\n",
+                TRACE("Created visible region with %lu rects\n", visRgn->rdh.nCount);
+                TRACE("  BoundingRect: %d, %d  %d, %d\n",
                        visRgn->rdh.rcBound.left, visRgn->rdh.rcBound.top,
                        visRgn->rdh.rcBound.right, visRgn->rdh.rcBound.bottom);
                 {
                     ULONG i;
                     for (i = 0; i < visRgn->rdh.nCount; i++)
                     {
-                        DPRINT("  Rect #%lu: %ld,%ld  %ld,%ld\n", i+1,
+                        TRACE("  Rect #%lu: %ld,%ld  %ld,%ld\n", i+1,
                                visRgn->Buffer[i].left, visRgn->Buffer[i].top,
                                visRgn->Buffer[i].right, visRgn->Buffer[i].bottom);
                     }
@@ -92,13 +90,13 @@ IntEngWndUpdateClipObj(
         }
         else
         {
-            DPRINT1("Warning: Couldn't lock visible region of window DC\n");
+            WARN("Couldn't lock visible region of window DC\n");
         }
         GreDeleteObject(hVisRgn);
     }
     else
     {
-        DPRINT1("Warning: VIS_ComputeVisibleRegion failed!\n");
+        WARN("VIS_ComputeVisibleRegion failed!\n");
     }
 
     if (ClipObj == NULL)
@@ -110,7 +108,7 @@ IntEngWndUpdateClipObj(
 
     if (ClipObj == NULL)
     {
-        DPRINT1("Warning: IntEngCreateClipRegion() failed!\n");
+        ERR("IntEngCreateClipRegion() failed!\n");
         return FALSE;
     }
 
@@ -192,7 +190,7 @@ EngCreateWnd(
     BOOL calledFromUser;
     DECLARE_RETURN(WNDOBJ*);
 
-    DPRINT1("EngCreateWnd: pso = 0x%p, hwnd = 0x%p, pfn = 0x%p, fl = 0x%lx, pixfmt = %d\n",
+    TRACE("EngCreateWnd: pso = 0x%p, hwnd = 0x%p, pfn = 0x%p, fl = 0x%lx, pixfmt = %d\n",
             pso, hWnd, pfn, fl, iPixelFormat);
 
     calledFromUser = UserIsEntered();
@@ -211,7 +209,7 @@ EngCreateWnd(
     WndObjInt = EngAllocMem(0, sizeof (WNDGDI), GDITAG_WNDOBJ);
     if (WndObjInt == NULL)
     {
-        DPRINT1("Failed to allocate memory for a WND structure!\n");
+        ERR("Failed to allocate memory for a WND structure!\n");
         RETURN( NULL);
     }
 
@@ -238,7 +236,7 @@ EngCreateWnd(
     IntSetProp(Window, AtomWndObj, WndObjInt);
     ++gcountPWO;
 
-    DPRINT("EngCreateWnd: SUCCESS!\n");
+    TRACE("EngCreateWnd: SUCCESS!\n");
 
     RETURN( WndObjUser);
 
@@ -264,7 +262,7 @@ EngDeleteWnd(
     PWND Window;
     BOOL calledFromUser;
 
-    DPRINT("EngDeleteWnd: pwo = 0x%p\n", pwo);
+    TRACE("EngDeleteWnd: pwo = 0x%p\n", pwo);
 
     calledFromUser = UserIsEntered();
     if (!calledFromUser) {
@@ -275,7 +273,7 @@ EngDeleteWnd(
     Window = UserGetWindowObject(WndObjInt->Hwnd);
     if (Window == NULL)
     {
-        DPRINT1("Warning: Couldnt get window object for WndObjInt->Hwnd!!!\n");
+        ERR("Couldnt get window object for WndObjInt->Hwnd!!!\n");
     }
     else
     {
@@ -307,10 +305,10 @@ WNDOBJ_bEnum(
     WNDGDI *WndObjInt = ObjToGDI(pwo, WND);
     BOOL Ret;
 
-    DPRINT("WNDOBJ_bEnum: pwo = 0x%p, cj = %lu, pul = 0x%p\n", pwo, cj, pul);
+    TRACE("WNDOBJ_bEnum: pwo = 0x%p, cj = %lu, pul = 0x%p\n", pwo, cj, pul);
     Ret = CLIPOBJ_bEnum(WndObjInt->ClientClipObj, cj, pul);
 
-    DPRINT("WNDOBJ_bEnum: Returning %s\n", Ret ? "True" : "False");
+    TRACE("WNDOBJ_bEnum: Returning %s\n", Ret ? "True" : "False");
     return Ret;
 }
 
@@ -329,13 +327,13 @@ WNDOBJ_cEnumStart(
     WNDGDI *WndObjInt = ObjToGDI(pwo, WND);
     ULONG Ret;
 
-    DPRINT("WNDOBJ_cEnumStart: pwo = 0x%p, iType = %lu, iDirection = %lu, cLimit = %lu\n",
+    TRACE("WNDOBJ_cEnumStart: pwo = 0x%p, iType = %lu, iDirection = %lu, cLimit = %lu\n",
            pwo, iType, iDirection, cLimit);
 
     /* FIXME: Should we enumerate all rectangles or not? */
     Ret = CLIPOBJ_cEnumStart(WndObjInt->ClientClipObj, FALSE, iType, iDirection, cLimit);
 
-    DPRINT("WNDOBJ_cEnumStart: Returning 0x%lx\n", Ret);
+    TRACE("WNDOBJ_cEnumStart: Returning 0x%lx\n", Ret);
     return Ret;
 }
 
@@ -351,7 +349,7 @@ WNDOBJ_vSetConsumer(
 {
     BOOL Hack;
 
-    DPRINT("WNDOBJ_vSetConsumer: pwo = 0x%p, pvConsumer = 0x%p\n", pwo, pvConsumer);
+    TRACE("WNDOBJ_vSetConsumer: pwo = 0x%p, pvConsumer = 0x%p\n", pwo, pvConsumer);
 
     Hack = (pwo->pvConsumer == NULL);
     pwo->pvConsumer = pvConsumer;
@@ -366,6 +364,7 @@ WNDOBJ_vSetConsumer(
      */
     if (Hack)
     {
+        FIXME("Is this hack really needed?\n");
         IntEngWndCallChangeProc(pwo, WOC_RGN_CLIENT);
         IntEngWndCallChangeProc(pwo, WOC_CHANGED);
         IntEngWndCallChangeProc(pwo, WOC_DRAWN);