Changeset 273 for trunk/src/3rdparty/os2/xsystray/xsystray.c

Timestamp:

Nov 2, 2009, 3:10:29 AM (16 years ago)

Author:

Dmitry A. Kuminov

Message:

3rdparty: os2/xsystray: Use custom shared memory pool for structures posted by the server to the client windows. Process mouse/wheel and context menu messages in the icon area and post them to the respective client windows. Use smaller spacing between icons (one pad unit instead of two).

File:

• trunk/src/3rdparty/os2/xsystray/xsystray.c (modified) (30 diffs)

trunk/src/3rdparty/os2/xsystray/xsystray.c

@@ -111 +111 @@
  */
 
-typedef struct _ICONDATA
+typedef struct
 {
     HWND        hwnd;
                 // associated window
-    ULONG       ulId;
+    UId;
                 // icon ID
     HPOINTER    hIcon;
@@ -123 +123 @@
     PSZ         pszToolTip;
                 // icon tooltip (NULL if none)
+
+
+
 
 } ICONDATA, *PICONDATA;
@@ -147 +150 @@
     size_t      cIconsMax;
                 // maximum number of icons pIcons can fit
+
+
 
 } SYSTRAYDATA, *PSYSTRAYDATA;
@@ -154 +159 @@
         // space for the newly added icon
 
-static ULONG QWL_USER_SERVER_DATA = 0;
-             // offset to the PXCENTERWIDGET pointer in the widget data array
+#define SERVER_MEMORYPOOL_SIZE 65536
+        // taking NOTIFYDATA size into account (<=32 B), this is enough for at
+        // least 2048 simultaneous notification messages, which (even taking
+        // slowly responsing clients into account) sounds sane since in most
+        // cases the structure is freed once it reaches the target event queue
+        // and before a message created as a copy of it is sent to the target
+        // window procedure
 
 #define TID_CHECKALIVE          1
@@ -161 +171 @@
 #define TID_CHECKALIVE_TIMEOUT  2000 // ms
          // how often to perform alive checks
+
+
+
+
+
+
+
+
 
 static
@@ -299 +317 @@
 {
     pData->hwnd = NULLHANDLE;
-    pData->ulId = 0;
+    pData->uId = 0;
     if (pData->hIcon != NULLHANDLE)
     {
@@ -323 +341 @@
 PICONDATA FindIconData(PSYSTRAYDATA pSysTrayData,
                        HWND hwnd,       // in: associated window handle
-                       ULONG ulId,      // in: icon ID
+                       U     // in: icon ID
                        size_t *pIdx)    // out: index of the icon in the icon array
                                         // (optional, may be NULL)
@@ -331 +349 @@
     {
         if (pSysTrayData->pIcons[i].hwnd == hwnd &&
-            pSysTrayData->pIcons[i].ulId == ulId)
+            pSysTrayData->pIcons[i].uId)
         {
             if (pIdx)
@@ -342 +360 @@
         *pIdx = i;
     return NULL;
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
 }
 
@@ -404 +541 @@
             {
                 PSIZEL pszl = (PSIZEL)mp2;
-                LONG pad2 = pSysTrayData->lIconPad * 2;
-                pszl->cx = (pSysTrayData->lIconWidth + pad2) * pSysTrayData->cIcons; // desired width
-                pszl->cy = pSysTrayData->lIconHeight + pad2; // desired minimum height
+                LONG pad = pSysTrayData->lIconPad;
+                size_t cnt = pSysTrayData->cIcons;
+                // desired width
+                if (cnt)
+                    pszl->cx = pad + (pSysTrayData->lIconWidth + pad) * cnt;
+                else
+                    pszl->cx = 0;
+                // desired minimum height
+                pszl->cy = pSysTrayData->lIconHeight + pad * 2;
                 brc = TRUE;
             }
@@ -421 +564 @@
  *      implementation for WM_PAINT in fnwpXSysTray.
  *
- *      This really does nothing, except painting a
- *      3D rectangle and printing a question mark.
+ *      Draws all the icons. If the widget's center is located to the left from
+ *      the XCenter's center, icons go left to right. Otherwise, they go right
+ *      to left.
+ */
+/*
+        +---------------------------+  p = lIconPad
+        |     p                     |  w = lIconWidth
+        |   +-------+   +-------+   |  h = lIconHeight
+        | p |   w   | p |   w   | p |
+        |   |      h|   |      h|   |
+        |   |       |   |       |   |  If "Frame around statics" is on in XCenter
+        |   +-------+   +-------+   |  properties, then a 1 px 3D frame is drawn
+        |     p                     |  within the pad area. So, lIconPad must
+        +---------------------------+  be at least 2 px.
  */
 
@@ -438 +593 @@
         RECTL   rcl;
         BOOL    bLeftToRight;
-        LONG    x, y, lTotalWidth;
+        LONG    x, y, l;
         size_t  i;
 
@@ -481 +636 @@
         // always center the icon vertically (we may be given more height than
         // we requested)
-        y = (swp.cy - pSysTrayData->lIconWidth) / 2;
+        y = (swp.cy - pSysTrayData->lIcon) / 2;
 
         if (bLeftToRight)
@@ -488 +643 @@
             x = swp.cx - pSysTrayData->lIconPad - pSysTrayData->lIconWidth;
 
-        lTotalWidth = pSysTrayData->lIconWidth + pSysTrayData->lIconPad * 2;
+        l;
 
         // where to start from?
         if (bLeftToRight)
         {
-            i = rclPaint.xLeft / lTotalWidth;
-            x = pSysTrayData->lIconPad + i * lTotalWidth;
+            i = rclPaint.xLeft / l;
+            x = pSysTrayData->lIconPad + i * l;
         }
         else
         {
-            i = (swp.cx - rclPaint.xRight) / lTotalWidth;
-            x = swp.cx - (i + 1) * lTotalWidth + pSysTrayData->lIconPad;
+            i = (swp.cx - rclPaint.xRight) / lIconStep;
+            x = swp.cx - (i + 1) * lIconStep;
+            // negate the step, for convenience
+            lIconStep = -lIconStep;
         }
 
         // draw as many icons as we can / need
-        for (i = 0; i < pSysTrayData->cIcons; ++i)
+        for (; i < pSysTrayData->cIcons; ++i)
         {
             if (x >= rclPaint.xRight)
@@ -509 +666 @@
 
             DrawPointer(hps, x, y, pSysTrayData->pIcons[i].hIcon, DP_MINI);
-            if (bLeftToRight)
-                x += lTotalWidth;
-            else
-                x -= lTotalWidth;
+            x += lIconStep;
         }
 
         WinEndPaint(hps);
     }
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
 }
 
@@ -562 +854 @@
 
             PSYSTRAYDATA pSysTrayData = NULL;
+
 
             WinSetWindowPtr(hwnd, QWL_USER, mp1);
@@ -575 +868 @@
 
             // initialize the SYSTRAYDATA structure
+
             pSysTrayData->lIconWidth = WinQuerySysValue(HWND_DESKTOP, SV_CXICON) / 2;
             pSysTrayData->lIconHeight = WinQuerySysValue(HWND_DESKTOP, SV_CYICON) / 2;
@@ -583 +877 @@
             if (pSysTrayData->pIcons == NULL)
             {
-                free(pSysTrayData);
+                (pSysTrayData);
                 return (MRESULT)TRUE;
             }
             pSysTrayData->cIcons = 0;
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
 
             // create the "server" window (note that we pass the XCENTERWIDGET
@@ -598 +918 @@
             if (pSysTrayData->hwndServer == NULLHANDLE)
             {
-                free(pSysTrayData->pIcons);
-                free(pSysTrayData);
+                FreeSysTrayData(pSysTrayData);
                 return (MRESULT)TRUE;
             }
@@ -606 +925 @@
 
             // inform all interested parties that we are fired up
-            // (NOTE: keep in sync with xstGetSysTrayCreatedMsgId())
-            WinBroadcastMsg(HWND_DESKTOP,
-                            WinAddAtom(WinQuerySystemAtomTable(),
-                                       "ExtendedSysTray.WM_XST_CREATED"),
-                            NULL, NULL,
-                            BMSG_POST);
+            WinBroadcastMsg(HWND_DESKTOP, WM_XST_CREATED,
+                            NULL, NULL, BMSG_POST);
 
             return FALSE; // confirm success
@@ -628 +943 @@
 
             PSYSTRAYDATA pSysTrayData = (PSYSTRAYDATA)pWidget->pUser;
-            size_t i;
-
-            // destroy the server
-            WinDestroyWindow(pSysTrayData->hwndServer);
-            pSysTrayData->hwndServer = NULLHANDLE;
-
-            // free all system tray data
-            for (i = 0; i < pSysTrayData->cIcons; ++i)
-                FreeIconData(&pSysTrayData->pIcons[i]);
-            pSysTrayData->cIcons = 0;
-            free(pSysTrayData->pIcons);
-            pSysTrayData->pIcons = NULL;
-
-            // make sure we remove the check alive timer
-            WgtXSysTrayUpdateAfterIconAddRemove(pWidget);
-
-            free(pSysTrayData);
+
+            // stop the check alive timer
+            WinStopTimer(pWidget->habWidget, pSysTrayData->hwndServer,
+                         TID_CHECKALIVE);
+
+            FreeSysTrayData(pSysTrayData);
             pWidget->pUser = NULL;
 
@@ -680 +984 @@
         break; */
 
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
         default:
             break;
@@ -713 +1046 @@
     }
 
-    if (pSysTrayData->pIcons != NULL)
-    {
-        // ask XCenter to take our new size into account (this will also
-        // invalidate us). If pIcons is NULL it means that we are in WM_DESTROY,
-        // in which case XCenter will do everything for us
-        WinPostMsg(pWidget->pGlobals->hwndClient,
-                   XCM_REFORMAT,
-                   (MPARAM)XFMF_GETWIDGETSIZES,
-                   0);
-    }
+    // ask XCenter to take our new size into account (this will also
+    // invalidate us)
+    WinPostMsg(pWidget->pGlobals->hwndClient,
+               XCM_REFORMAT,
+               (MPARAM)XFMF_GETWIDGETSIZES,
+               0);
 }
 
@@ -767 +1096 @@
             LOGF(("SYSTRAYCMD_ADDICON\n"));
             LOGF((" hwnd  %x\n", pCtlData->hwndSender));
-            LOGF((" ulId  %ld\n", pCtlData->u.icon.ulId));
+            LOGF((" uId));
             LOGF((" hIcon %x\n", pCtlData->u.icon.hIcon));
 
@@ -785 +1114 @@
 
             pData = FindIconData(pSysTrayData, pCtlData->hwndSender,
-                                 pCtlData->u.icon.ulId, &i);
+                                 pCtlData->u.icon.uId, &i);
             if (pData)
             {
@@ -825 +1154 @@
 
                 pData = &pSysTrayData->pIcons[i];
+
+
                 pData->hwnd = pCtlData->hwndSender;
-                pData->ulId = pCtlData->u.icon.ulId;
+                pData->uId;
                 pData->hIcon = hIcon;
                 pData->ulMsgId = pCtlData->u.icon.ulMsgId;
@@ -844 +1175 @@
             LOGF(("SYSTRAYCMD_REMOVEICON\n"));
             LOGF((" hwnd  %x\n", pCtlData->hwndSender));
-            LOGF((" ulId  %ld\n", pCtlData->u.icon.ulId));
+            LOGF((" uId));
 
             pCtlData->bAcknowledged = TRUE;
 
             pData = FindIconData(pSysTrayData, pCtlData->hwndSender,
-                                 pCtlData->u.icon.ulId, &i);
+                                 pCtlData->u.icon.uId, &i);
             if (pData)
             {
@@ -893 +1224 @@
 {
     PSYSTRAYDATA pSysTrayData = (PSYSTRAYDATA)pWidget->pUser;
+
+
+
 
     if (usTimerId == TID_CHECKALIVE)
@@ -904 +1238 @@
             if (!WinIsWindow(pWidget->habWidget, pSysTrayData->pIcons[i].hwnd))
             {
+
+
                 LOGF(("Removing icon of dead window!\n"));
-                LOGF((" hwnd  %x\n", pSysTrayData->pIcons[i].hwnd));
-                LOGF((" ulId  %ld\n", pSysTrayData->pIcons[i].ulId));
-                LOGF((" hIcon %x\n", pSysTrayData->pIcons[i].hIcon));
+                LOGF((" hwnd  %x\n", pData->hwnd));
+                LOGF((" usId  %ld\n", pData->usId));
+                LOGF((" hIcon %x\n", pData->hIcon));
+
+                // free memory blocks from the pool allocated for this client
+                ulMemPoolMax = pMemPoolHdr->ulBeyond;
+                if (ulMemPoolMax > pMemPoolHdr->ulNeedsCommit)
+                    ulMemPoolMax = pMemPoolHdr->ulNeedsCommit;
+                ulMemPoolMax -= sizeof(MEMPOOLBLK);
+
+                pMemPoolBlk = pMemPoolHdr->aBlocks;
+                while ((ULONG)pMemPoolBlk <= ulMemPoolMax)
+                {
+                    if (pMemPoolBlk->hwnd == pData->hwnd)
+                    {
+                        LOGF((" freeing memory block %p\n", pMemPoolBlk));
+                        __atomic_cmpxchg32((uint32_t *)&pMemPoolBlk->hwnd,
+                                           NULLHANDLE, hwnd);
+                    }
+                    ++pMemPoolBlk;
+                }
 
                 bAnyDead = TRUE;
-                FreeIconData(&pSysTrayData->pIcons[i]);
-                // hwnd is NULLHANDLE here
+                FreeIconData();
+                // hwnd is NULLHANDLE here
             }
         }
@@ -1128 +1482 @@
         }
 
+
+
+
+
+
+
+
         // no error:
         // return widget classes array