[WIN32K]
authorJérôme Gardou <jerome.gardou@reactos.org>
Wed, 29 Jan 2014 19:09:15 +0000 (19:09 +0000)
committerJérôme Gardou <jerome.gardou@reactos.org>
Wed, 29 Jan 2014 19:09:15 +0000 (19:09 +0000)
 - Formatting, no code change.

svn path=/trunk/; revision=61884

reactos/win32ss/gdi/eng/eng.h
reactos/win32ss/gdi/eng/engwindow.c

index fba140b..fac3f97 100644 (file)
@@ -18,8 +18,8 @@ IntEngMaskBlt(SURFOBJ *psoDest,
 
 VOID FASTCALL
 IntEngWindowChanged(
-        PWND Window,
-        FLONG flChanged);
+    _In_    PWND Window,
+    _In_    FLONG flChanged);
 
 VOID FASTCALL IntGdiAcquireSemaphore ( HSEMAPHORE hsem );
 VOID FASTCALL IntGdiReleaseSemaphore ( HSEMAPHORE hsem );
index cbb7f7c..423f61f 100644 (file)
@@ -6,9 +6,6 @@
  * PROGRAMER:         Gregor Anich
  */
 
-/* TODO: Check how the WNDOBJ implementation should behave with a driver on windows.
- */
-
 #include <win32k.h>
 
 #define NDEBUG
@@ -22,32 +19,32 @@ INT gcountPWO = 0;
 VOID
 FASTCALL
 IntEngWndCallChangeProc(
-  IN WNDOBJ *pwo,
-  IN FLONG   flChanged)
+    IN WNDOBJ *pwo,
+    IN FLONG   flChanged)
 {
-  WNDGDI *WndObjInt = ObjToGDI(pwo, WND);
+    WNDGDI *WndObjInt = ObjToGDI(pwo, WND);
 
-  if (WndObjInt->ChangeProc == NULL)
+    if (WndObjInt->ChangeProc == NULL)
     {
-      return;
+        return;
     }
 
-  /* check flags of the WNDOBJ */
-  flChanged &= WndObjInt->Flags;
-  if (flChanged == 0)
+    /* check flags of the WNDOBJ */
+    flChanged &= WndObjInt->Flags;
+    if (flChanged == 0)
     {
-      return;
+        return;
     }
 
-  /* Call the WNDOBJCHANGEPROC */
-  if (flChanged == WOC_CHANGED)
+    /* Call the WNDOBJCHANGEPROC */
+    if (flChanged == WOC_CHANGED)
     {
-      pwo = NULL;
+        pwo = NULL;
     }
 
-  DPRINT("Calling WNDOBJCHANGEPROC (0x%p), Changed = 0x%x\n",
-         WndObjInt->ChangeProc, flChanged);
-  WndObjInt->ChangeProc(pwo, flChanged);
+    DPRINT("Calling WNDOBJCHANGEPROC (0x%p), Changed = 0x%x\n",
+           WndObjInt->ChangeProc, flChanged);
+    WndObjInt->ChangeProc(pwo, flChanged);
 }
 
 /*
@@ -56,74 +53,74 @@ IntEngWndCallChangeProc(
 BOOLEAN
 FASTCALL
 IntEngWndUpdateClipObj(
-  WNDGDI *WndObjInt,
-  PWND Window)
+    WNDGDI *WndObjInt,
+    PWND Window)
 {
-  HRGN hVisRgn;
-  PROSRGNDATA visRgn;
-  CLIPOBJ *ClipObj = NULL;
-  CLIPOBJ *OldClipObj;
-
-  DPRINT("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)
+    HRGN hVisRgn;
+    PROSRGNDATA visRgn;
+    CLIPOBJ *ClipObj = NULL;
+    CLIPOBJ *OldClipObj;
+
+    DPRINT("IntEngWndUpdateClipObj\n");
+
+    hVisRgn = VIS_ComputeVisibleRegion(Window, TRUE, TRUE, TRUE);
+    if (hVisRgn != NULL)
     {
-      if (visRgn->rdh.nCount > 0)
-      {
-        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",
-               visRgn->rdh.rcBound.left, visRgn->rdh.rcBound.top,
-               visRgn->rdh.rcBound.right, visRgn->rdh.rcBound.bottom);
+        NtGdiOffsetRgn(hVisRgn, Window->rcClient.left, Window->rcClient.top);
+        visRgn = RGNOBJAPI_Lock(hVisRgn, NULL);
+        if (visRgn != NULL)
+        {
+            if (visRgn->rdh.nCount > 0)
+            {
+                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",
+                       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,
+                               visRgn->Buffer[i].left, visRgn->Buffer[i].top,
+                               visRgn->Buffer[i].right, visRgn->Buffer[i].bottom);
+                    }
+                }
+            }
+            RGNOBJAPI_Unlock(visRgn);
+        }
+        else
         {
-          ULONG i;
-          for (i = 0; i < visRgn->rdh.nCount; i++)
-          {
-            DPRINT("  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);
-          }
+            DPRINT1("Warning: Couldn't lock visible region of window DC\n");
         }
-      }
-      RGNOBJAPI_Unlock(visRgn);
+        GreDeleteObject(hVisRgn);
     }
     else
     {
-      DPRINT1("Warning: Couldn't lock visible region of window DC\n");
+        DPRINT1("Warning: VIS_ComputeVisibleRegion failed!\n");
     }
-    GreDeleteObject(hVisRgn);
-  }
-  else
-  {
-    DPRINT1("Warning: VIS_ComputeVisibleRegion failed!\n");
-  }
-
-  if (ClipObj == NULL)
-  {
-    /* Fall back to client rect */
-    ClipObj = IntEngCreateClipRegion(1, &Window->rcClient,
-                                     &Window->rcClient);
-  }
-
-  if (ClipObj == NULL)
-  {
-    DPRINT1("Warning: IntEngCreateClipRegion() failed!\n");
-    return FALSE;
-  }
-
-  RtlCopyMemory(&WndObjInt->WndObj.coClient, ClipObj, sizeof (CLIPOBJ));
-  RtlCopyMemory(&WndObjInt->WndObj.rclClient, &Window->rcClient, sizeof (RECT));
-  OldClipObj = InterlockedExchangePointer((PVOID*)&WndObjInt->ClientClipObj, ClipObj);
-  if (OldClipObj != NULL)
-    IntEngDeleteClipRegion(OldClipObj);
-
-  return TRUE;
+
+    if (ClipObj == NULL)
+    {
+        /* Fall back to client rect */
+        ClipObj = IntEngCreateClipRegion(1, &Window->rcClient,
+                                         &Window->rcClient);
+    }
+
+    if (ClipObj == NULL)
+    {
+        DPRINT1("Warning: IntEngCreateClipRegion() failed!\n");
+        return FALSE;
+    }
+
+    RtlCopyMemory(&WndObjInt->WndObj.coClient, ClipObj, sizeof (CLIPOBJ));
+    RtlCopyMemory(&WndObjInt->WndObj.rclClient, &Window->rcClient, sizeof (RECT));
+    OldClipObj = InterlockedExchangePointer((PVOID*)&WndObjInt->ClientClipObj, ClipObj);
+    if (OldClipObj != NULL)
+        IntEngDeleteClipRegion(OldClipObj);
+
+    return TRUE;
 }
 
 /*
@@ -132,49 +129,49 @@ IntEngWndUpdateClipObj(
 VOID
 FASTCALL
 IntEngWindowChanged(
-  PWND  Window,
-  FLONG           flChanged)
+    _In_    PWND  Window,
+    _In_    FLONG flChanged)
 {
-  PPROPERTY pprop;
-  WNDGDI *Current;
-  HWND hWnd;
-
-  ASSERT_IRQL_LESS_OR_EQUAL(PASSIVE_LEVEL);
-
-  hWnd = Window->head.h;
-  pprop = IntGetProp(Window, AtomWndObj);
-  if (!pprop)
-  {
-     return;
-  }
-  Current = (WNDGDI *)pprop->Data;
-  if ( gcountPWO &&
-       Current &&
-       Current->Hwnd == hWnd &&
-       Current->WndObj.pvConsumer != NULL )
-  {
-     /* Update the WNDOBJ */
-     switch (flChanged)
-     {
+    PPROPERTY pprop;
+    WNDGDI *Current;
+    HWND hWnd;
+
+    ASSERT_IRQL_LESS_OR_EQUAL(PASSIVE_LEVEL);
+
+    hWnd = Window->head.h;
+    pprop = IntGetProp(Window, AtomWndObj);
+    if (!pprop)
+    {
+        return;
+    }
+    Current = (WNDGDI *)pprop->Data;
+    if ( gcountPWO &&
+            Current &&
+            Current->Hwnd == hWnd &&
+            Current->WndObj.pvConsumer != NULL )
+    {
+        /* Update the WNDOBJ */
+        switch (flChanged)
+        {
         case WOC_RGN_CLIENT:
-        /* Update the clipobj and client rect of the WNDOBJ */
-           IntEngWndUpdateClipObj(Current, Window);
-           break;
+            /* Update the clipobj and client rect of the WNDOBJ */
+            IntEngWndUpdateClipObj(Current, Window);
+            break;
 
         case WOC_DELETE:
-        /* FIXME: Should the WNDOBJs be deleted by win32k or by the driver? */
-           break;
-     }
-
-     /* Call the change proc */
-     IntEngWndCallChangeProc(&Current->WndObj, flChanged);
-
-     /* HACK: Send WOC_CHANGED after WOC_RGN_CLIENT */
-     if (flChanged == WOC_RGN_CLIENT)
-     {
-        IntEngWndCallChangeProc(&Current->WndObj, WOC_CHANGED);
-     }
-  }
+            /* FIXME: Should the WNDOBJs be deleted by win32k or by the driver? */
+            break;
+        }
+
+        /* Call the change proc */
+        IntEngWndCallChangeProc(&Current->WndObj, flChanged);
+
+        /* HACK: Send WOC_CHANGED after WOC_RGN_CLIENT */
+        if (flChanged == WOC_RGN_CLIENT)
+        {
+            IntEngWndCallChangeProc(&Current->WndObj, WOC_CHANGED);
+        }
+    }
 }
 
 /*
@@ -183,75 +180,75 @@ IntEngWindowChanged(
 WNDOBJ*
 APIENTRY
 EngCreateWnd(
-  SURFOBJ          *pso,
-  HWND              hWnd,
-  WNDOBJCHANGEPROC  pfn,
-  FLONG             fl,
-  int               iPixelFormat)
+    SURFOBJ          *pso,
+    HWND              hWnd,
+    WNDOBJCHANGEPROC  pfn,
+    FLONG             fl,
+    int               iPixelFormat)
 {
-  WNDGDI *WndObjInt = NULL;
-  WNDOBJ *WndObjUser = NULL;
-  PWND Window;
-  BOOL calledFromUser;
-  DECLARE_RETURN(WNDOBJ*);
-
-  DPRINT("EngCreateWnd: pso = 0x%p, hwnd = 0x%p, pfn = 0x%p, fl = 0x%lx, pixfmt = %d\n",
-         pso, hWnd, pfn, fl, iPixelFormat);
-
-  calledFromUser = UserIsEntered();
-  if (!calledFromUser){
-     UserEnterShared();
-  }
-
-  /* Get window object */
-  Window = UserGetWindowObject(hWnd);
-  if (Window == NULL)
+    WNDGDI *WndObjInt = NULL;
+    WNDOBJ *WndObjUser = NULL;
+    PWND Window;
+    BOOL calledFromUser;
+    DECLARE_RETURN(WNDOBJ*);
+
+    DPRINT1("EngCreateWnd: pso = 0x%p, hwnd = 0x%p, pfn = 0x%p, fl = 0x%lx, pixfmt = %d\n",
+            pso, hWnd, pfn, fl, iPixelFormat);
+
+    calledFromUser = UserIsEntered();
+    if (!calledFromUser) {
+        UserEnterShared();
+    }
+
+    /* Get window object */
+    Window = UserGetWindowObject(hWnd);
+    if (Window == NULL)
     {
-      RETURN( NULL);
+        RETURN( NULL);
     }
 
-  /* Create WNDOBJ */
-  WndObjInt = EngAllocMem(0, sizeof (WNDGDI), GDITAG_WNDOBJ);
-  if (WndObjInt == NULL)
+    /* Create WNDOBJ */
+    WndObjInt = EngAllocMem(0, sizeof (WNDGDI), GDITAG_WNDOBJ);
+    if (WndObjInt == NULL)
     {
-      DPRINT1("Failed to allocate memory for a WND structure!\n");
-      RETURN( NULL);
+        DPRINT1("Failed to allocate memory for a WND structure!\n");
+        RETURN( NULL);
     }
 
-  /* Fill the clipobj */
-  WndObjInt->ClientClipObj = NULL;
-  if (!IntEngWndUpdateClipObj(WndObjInt, Window))
+    /* Fill the clipobj */
+    WndObjInt->ClientClipObj = NULL;
+    if (!IntEngWndUpdateClipObj(WndObjInt, Window))
     {
-      EngFreeMem(WndObjInt);
-      RETURN( NULL);
+        EngFreeMem(WndObjInt);
+        RETURN( NULL);
     }
 
-  /* Fill user object */
-  WndObjUser = GDIToObj(WndObjInt, WND);
-  WndObjUser->psoOwner = pso;
-  WndObjUser->pvConsumer = NULL;
+    /* Fill user object */
+    WndObjUser = GDIToObj(WndObjInt, WND);
+    WndObjUser->psoOwner = pso;
+    WndObjUser->pvConsumer = NULL;
 
-  /* Fill internal object */
-  WndObjInt->Hwnd = hWnd;
-  WndObjInt->ChangeProc = pfn;
-  WndObjInt->Flags = fl;
-  WndObjInt->PixelFormat = iPixelFormat;
+    /* Fill internal object */
+    WndObjInt->Hwnd = hWnd;
+    WndObjInt->ChangeProc = pfn;
+    WndObjInt->Flags = fl;
+    WndObjInt->PixelFormat = iPixelFormat;
 
-  /* associate object with window */
-  IntSetProp(Window, AtomWndObj, WndObjInt);
-  ++gcountPWO;
+    /* associate object with window */
+    IntSetProp(Window, AtomWndObj, WndObjInt);
+    ++gcountPWO;
 
-  DPRINT("EngCreateWnd: SUCCESS!\n");
+    DPRINT("EngCreateWnd: SUCCESS!\n");
 
-  RETURN( WndObjUser);
+    RETURN( WndObjUser);
 
 CLEANUP:
 
-  if (!calledFromUser){
-    UserLeave();
-  }
+    if (!calledFromUser) {
+        UserLeave();
+    }
 
-  END_CLEANUP;
+    END_CLEANUP;
 }
 
 
@@ -261,39 +258,39 @@ CLEANUP:
 VOID
 APIENTRY
 EngDeleteWnd(
-  IN WNDOBJ *pwo)
+    IN WNDOBJ *pwo)
 {
-  WNDGDI *WndObjInt = ObjToGDI(pwo, WND);
-  PWND Window;
-  BOOL calledFromUser;
-
-  DPRINT("EngDeleteWnd: pwo = 0x%p\n", pwo);
-
-  calledFromUser = UserIsEntered();
-  if (!calledFromUser){
-     UserEnterExclusive();
-  }
-
-  /* Get window object */
-  Window = UserGetWindowObject(WndObjInt->Hwnd);
-  if (Window == NULL)
-  {
-     DPRINT1("Warning: Couldnt get window object for WndObjInt->Hwnd!!!\n");
-  }
-  else
-  {
-    /* Remove object from window */
-    IntRemoveProp(Window, AtomWndObj);
-    --gcountPWO;
-  }
-
-  if (!calledFromUser){
-     UserLeave();
-  }
-
-  /* Free resources */
-  IntEngDeleteClipRegion(WndObjInt->ClientClipObj);
-  EngFreeMem(WndObjInt);
+    WNDGDI *WndObjInt = ObjToGDI(pwo, WND);
+    PWND Window;
+    BOOL calledFromUser;
+
+    DPRINT("EngDeleteWnd: pwo = 0x%p\n", pwo);
+
+    calledFromUser = UserIsEntered();
+    if (!calledFromUser) {
+        UserEnterExclusive();
+    }
+
+    /* Get window object */
+    Window = UserGetWindowObject(WndObjInt->Hwnd);
+    if (Window == NULL)
+    {
+        DPRINT1("Warning: Couldnt get window object for WndObjInt->Hwnd!!!\n");
+    }
+    else
+    {
+        /* Remove object from window */
+        IntRemoveProp(Window, AtomWndObj);
+        --gcountPWO;
+    }
+
+    if (!calledFromUser) {
+        UserLeave();
+    }
+
+    /* Free resources */
+    IntEngDeleteClipRegion(WndObjInt->ClientClipObj);
+    EngFreeMem(WndObjInt);
 }
 
 
@@ -303,18 +300,18 @@ EngDeleteWnd(
 BOOL
 APIENTRY
 WNDOBJ_bEnum(
-  IN WNDOBJ  *pwo,
-  IN ULONG  cj,
-  OUT ULONG  *pul)
+    IN WNDOBJ  *pwo,
+    IN ULONG  cj,
+    OUT ULONG  *pul)
 {
-  WNDGDI *WndObjInt = ObjToGDI(pwo, WND);
-  BOOL Ret;
+    WNDGDI *WndObjInt = ObjToGDI(pwo, WND);
+    BOOL Ret;
 
-  DPRINT("WNDOBJ_bEnum: pwo = 0x%p, cj = %lu, pul = 0x%p\n", pwo, cj, pul);
-  Ret = CLIPOBJ_bEnum(WndObjInt->ClientClipObj, cj, pul);
+    DPRINT("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");
-  return Ret;
+    DPRINT("WNDOBJ_bEnum: Returning %s\n", Ret ? "True" : "False");
+    return Ret;
 }
 
 
@@ -324,22 +321,22 @@ WNDOBJ_bEnum(
 ULONG
 APIENTRY
 WNDOBJ_cEnumStart(
-  IN WNDOBJ  *pwo,
-  IN ULONG  iType,
-  IN ULONG  iDirection,
-  IN ULONG  cLimit)
+    IN WNDOBJ  *pwo,
+    IN ULONG  iType,
+    IN ULONG  iDirection,
+    IN ULONG  cLimit)
 {
-  WNDGDI *WndObjInt = ObjToGDI(pwo, WND);
-  ULONG Ret;
+    WNDGDI *WndObjInt = ObjToGDI(pwo, WND);
+    ULONG Ret;
 
-  DPRINT("WNDOBJ_cEnumStart: pwo = 0x%p, iType = %lu, iDirection = %lu, cLimit = %lu\n",
-         pwo, iType, iDirection, cLimit);
+    DPRINT("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);
+    /* 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);
-  return Ret;
+    DPRINT("WNDOBJ_cEnumStart: Returning 0x%lx\n", Ret);
+    return Ret;
 }
 
 
@@ -349,29 +346,29 @@ WNDOBJ_cEnumStart(
 VOID
 APIENTRY
 WNDOBJ_vSetConsumer(
-  IN WNDOBJ  *pwo,
-  IN PVOID  pvConsumer)
+    IN WNDOBJ  *pwo,
+    IN PVOID  pvConsumer)
 {
-  BOOL Hack;
-
-  DPRINT("WNDOBJ_vSetConsumer: pwo = 0x%p, pvConsumer = 0x%p\n", pwo, pvConsumer);
-
-  Hack = (pwo->pvConsumer == NULL);
-  pwo->pvConsumer = pvConsumer;
-
-  /* HACKHACKHACK
-   *
-   * MSDN says that the WNDOBJCHANGEPROC will be called with the most recent state
-   * when a WNDOBJ is created - we do it here because most drivers will need pvConsumer
-   * in the callback to identify the WNDOBJ I think.
-   *
-   *  - blight
-   */
-  if (Hack)
+    BOOL Hack;
+
+    DPRINT("WNDOBJ_vSetConsumer: pwo = 0x%p, pvConsumer = 0x%p\n", pwo, pvConsumer);
+
+    Hack = (pwo->pvConsumer == NULL);
+    pwo->pvConsumer = pvConsumer;
+
+    /* HACKHACKHACK
+     *
+     * MSDN says that the WNDOBJCHANGEPROC will be called with the most recent state
+     * when a WNDOBJ is created - we do it here because most drivers will need pvConsumer
+     * in the callback to identify the WNDOBJ I think.
+     *
+     *  - blight
+     */
+    if (Hack)
     {
-      IntEngWndCallChangeProc(pwo, WOC_RGN_CLIENT);
-      IntEngWndCallChangeProc(pwo, WOC_CHANGED);
-      IntEngWndCallChangeProc(pwo, WOC_DRAWN);
+        IntEngWndCallChangeProc(pwo, WOC_RGN_CLIENT);
+        IntEngWndCallChangeProc(pwo, WOC_CHANGED);
+        IntEngWndCallChangeProc(pwo, WOC_DRAWN);
     }
 }