Changeset 273 for trunk

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).

Location:

trunk/src/3rdparty/os2/xsystray

Files:

• xsystray.c (modified) (30 diffs)
• xsystray.h (modified) (6 diffs)
• xsystray_api.c (modified) (16 diffs)
• xsystray_api.h (modified) (2 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

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

@@ -17 +17 @@
 #define XSYSTRAY_HEADER_INCLUDED
 
+
+
+
+
 #define XSYSTRAY_VERSION_MAJOR 0
 #define XSYSTRAY_VERSION_MINOR 1
@@ -26 +30 @@
 #define INTCLASS_WIDGET_XSYSTRAY        "ExtendedSysTray"
 #define HUMANSTR_WIDGET_XSYSTRAY        "Extended system tray"
+
+
+
 
 #define WM_XST_CONTROL  (WM_USER + 0)
@@ -38 +45 @@
     SYSTRAYCMD_HIDEBALLOON,
 } SYSTRAYCMD;
+
+
+
+
+
+
+
+
+
+
+
 
 typedef struct
@@ -58 +76 @@
         struct
         {
-            ULONG       ulId;
+            UId;
             HPOINTER    hIcon;
             ULONG       ulMsgId;
@@ -66 +84 @@
         struct
         {
-            ULONG   ulId;
+            UId;
             CHAR    szText[512];
         } tooltip;
@@ -77 +95 @@
 } SYSTRAYCTLDATA, *PSYSTRAYCTLDATA;
 
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
 #endif // XSYSTRAY_HEADER_INCLUDED
 

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

@@ -28 +28 @@
 #include <sys/builtin.h>        // atomics
 
-static HWND G_hwndSysTray = NULLHANDLE;
-            // window handle of the system tray server
-
-static PVOID G_pvMemoryPool = NULL;
-             // memory pool for SYSTRAYCTLDATA structs used by WM_XST_CONTROL
-             // messages. Note that once allocated, this memory is never freed:
-             // it is intentional since the memory is assumed to be always in
-             // need and that the system will free it when the application
-             // terminates
-
-#define MEMORYPOOL_SIZE 65536
-        // taking SYSTRAYCTLDATA size into account, this is enough for at least
-        // 64 threads sending WM_XST_CONTROL simultaneously, which sounds sane
+static
+volatile HWND G_hwndSysTray = NULLHANDLE;
+              // window handle of the system tray server
+
+static
+volatile PVOID G_pvMemoryPool = NULL;
+               // shared memory pool for SYSTRAYCTLDATA structs used by
+               // WM_XST_CONTROL messages. Note that once allocated, this memory
+               // is never freed: it is intentional since the memory is assumed
+               // to be always in need and that the system will free it when the
+               // application terminates
+
+#define CLIENT_MEMORYPOOL_SIZE 65536
+        // taking SYSTRAYCTLDATA size into account (<=1024 B), this is enough
+        // for at least 64 threads sending WM_XST_CONTROL simultaneously, which
+        // sounds sane
 
 // @todo to be on the safe side with casting in __atomic_cmpxchg32() we need
@@ -116 +119 @@
 }
 
-// This function allocates a SYSTRAYCTLDATA struct in the pool of shared memory.
-// If there is no free space in the pool, it returns NULL. The allocated memory
-// must be freed by FreeSysTrayCtlDataPtr() when not needed.
+/*
+ *@@ AllocSysTrayCtlDataPtr:
+ *      Allocates a SYSTRAYCTLDATA struct in the pool of shared memory.
+ *
+ *      If there is no free space in the pool, it returns NULL. The allocated
+ *      memory must be freed by FreeSysTrayCtlDataPtr() when not needed.
+ */
+
 static PSYSTRAYCTLDATA AllocSysTrayCtlDataPtr()
 {
@@ -128 +136 @@
     {
         // Note: we don't PAG_COMMIT, DosSubAllocMem will do so when needed
-        arc = DosAllocSharedMem((PVOID)&pvPool, NULL, MEMORYPOOL_SIZE,
+        arc = DosAllocSharedMem((PVOID)&pvPool, NULL, MEMORYPOOL_SIZE,
                                 PAG_READ | PAG_WRITE | OBJ_GIVEABLE);
         if (arc == NO_ERROR)
             arc = DosSubSetMem(pvPool,
                                DOSSUB_INIT | DOSSUB_SPARSE_OBJ,
-                               MEMORYPOOL_SIZE);
+                               MEMORYPOOL_SIZE);
         if (!__atomic_cmpxchg32((uint32_t *)&G_pvMemoryPool,
                                 (uint32_t)pvPool, (uint32_t)NULL))
@@ -149 +157 @@
     }
 
-    arc = DosSubAllocMem(G_pvMemoryPool, (PVOID)&pData, sizeof(SYSTRAYCTLDATA));
+    arc = DosSubAllocMem(G_pvMemoryPool, (PVOID)&pData, sizeof());
     if (arc != NO_ERROR)
         return NULL;
@@ -158 +166 @@
 static VOID FreeSysTrayCtlDataPtr(PSYSTRAYCTLDATA pData)
 {
-    DosSubFreeMem(G_pvMemoryPool, pData, sizeof(SYSTRAYCTLDATA));
+    DosSubFreeMem(G_pvMemoryPool, pData, sizeof());
 }
 
@@ -220 +228 @@
  *
  *          param1
- *              USHORT  usID        icon ID
- *              USHORT  usCode      notify code, one of XST_IN_ constants
+ *              USHORT  usID        icon ID
+ *              USHORT  us    notify code, one of XST_IN_ constants
  *
  *          param2
- *              PVOID   pData       notify code specific data (see below)
+ *              PVOID   pData       notify code specific data (see below)
  *
  *      The following notify codes are currently recognized:
@@ -232 +240 @@
  *              messages are recognized. param2 is a pointer to the XSTMOUSEMSG
  *              structure containing full mouse message details.
+
+
+
+
+
+
+
+
  */
 
 BOOL xstAddSysTrayIcon(HWND hwnd,       // in: window handle associated with the icon
-                       ULONG ulId,      // in: icon ID to add
+                       U     // in: icon ID to add
                        HPOINTER hIcon,  // in: icon handle
                        ULONG ulMsgId,   // in: message ID for notifications
@@ -252 +268 @@
         pData->ulCommand = SYSTRAYCMD_ADDICON;
         pData->hwndSender = hwnd;
-        pData->u.icon.ulId = ulId;
+        pData->u.icon.uId;
         pData->u.icon.hIcon = hIcon;
         pData->u.icon.ulMsgId = ulMsgId;
@@ -278 +294 @@
 
 BOOL xstRemoveSysTrayIcon(HWND hwnd,    // in: window handle associated with the icon
-                          ULONG ulId)   // in: icon ID to remove
+                          U  // in: icon ID to remove
 {
     BOOL brc;
@@ -287 +303 @@
     pData->ulCommand = SYSTRAYCMD_REMOVEICON;
     pData->hwndSender = hwnd;
-    pData->u.icon.ulId = ulId;
+    pData->u.icon.uId;
 
     brc = SendSysTrayCtlMsg(pData);
@@ -314 +330 @@
 
 BOOL xstSetSysTrayIconToolTip(HWND hwnd,    // in: window handle associated with the icon
-                              ULONG ulId,   // in: icon ID to set the tooltip for
+                              U  // in: icon ID to set the tooltip for
                               PSZ pszText)  // in: tooltip text
 {
@@ -324 +340 @@
     pData->ulCommand = SYSTRAYCMD_SETTOOLTIP;
     pData->hwndSender = hwnd;
-    pData->u.tooltip.ulId = ulId;
+    pData->u.tooltip.uId;
 
     if (pszText == NULL)
@@ -343 +359 @@
 }
 
-BOOL xstShowSysTrayIconBalloon(HWND hwnd, ULONG ulId, PSZ pszTitle, PSZ pszText,
+BOOL xstShowSysTrayIconBalloon(HWND hwnd, UId, PSZ pszTitle, PSZ pszText,
                                ULONG ulFlags, ULONG ulTimeout)
 {
@@ -350 +366 @@
 }
 
-BOOL xstHideSysTrayIconBalloon(HWND hwnd, ULONG ulId)
+BOOL xstHideSysTrayIconBalloon(HWND hwnd, UId)
 {
     // @todo implement
@@ -364 +380 @@
  *      Returns TRUE on success and FALSE otherwise.
  */
-BOOL xstQuerySysTrayIconRect(HWND hwnd, ULONG ulId, PRECTL prclRect)
+BOOL xstQuerySysTrayIconRect(HWND hwnd, UId, PRECTL prclRect)
 {
     // @todo implement
@@ -384 +400 @@
 ULONG xstGetSysTrayCreatedMsgId()
 {
-    // NOTE: keep in sync with fnwpXSysTray()::WM_CREATED
-
     static ULONG WM_XST_CREATED = 0;
     if (WM_XST_CREATED == 0)
         WM_XST_CREATED = WinAddAtom(WinQuerySystemAtomTable(),
-                                    "ExtendedSysTray.WM_XST_CREATED");
+                                    );
     return WM_XST_CREATED;
 }

trunk/src/3rdparty/os2/xsystray/xsystray_api.h

@@ -24 +24 @@
 // tray (refer to xstAddSysTrayIcon() for details)
 #define XST_IN_MOUSE    0x0001
+
+
 
 // structure for XST_IN_MOUSE
@@ -29 +31 @@
 {
     ULONG  ulMouseMsg;
+
     POINTS ptsPointerPos;
+
     USHORT fsHitTestRes;
+
     USHORT fsFlags;
+
 
 } XSTMOUSEMSG, *PXSTMOUSEMSG;
 
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
 BOOL xstQuerySysTrayVersion(PULONG pulMajor, PULONG pulMinor, PULONG pulRevision);
-BOOL xstAddSysTrayIcon(HWND hwnd, ULONG ulId, HPOINTER hIcon, ULONG ulMsgId,
+BOOL xstAddSysTrayIcon(HWND hwnd, UId, HPOINTER hIcon, ULONG ulMsgId,
                        ULONG ulFlags);
-BOOL xstRemoveSysTrayIcon(HWND hwnd, ULONG ulId);
-BOOL xstSetSysTrayIconToolTip(HWND hwnd, ULONG ulId, PSZ pszText);
-BOOL xstShowSysTrayIconBalloon(HWND hwnd, ULONG ulId, PSZ pszTitle, PSZ pszText,
+BOOL xstRemoveSysTrayIcon(HWND hwnd, UId);
+BOOL xstSetSysTrayIconToolTip(HWND hwnd, UId, PSZ pszText);
+BOOL xstShowSysTrayIconBalloon(HWND hwnd, UId, PSZ pszTitle, PSZ pszText,
                                ULONG ulFlags, ULONG ulTimeout);
-BOOL xstHideSysTrayIconBalloon(HWND hwnd, ULONG ulId);
+BOOL xstHideSysTrayIconBalloon(HWND hwnd, UId);
 
-BOOL xstQuerySysTrayIconRect(HWND hwnd, ULONG ulId, PRECTL prclRect);
+BOOL xstQuerySysTrayIconRect(HWND hwnd, UId, PRECTL prclRect);
 
 ULONG xstGetSysTrayCreatedMsgId();