Changeset 256

Timestamp:

Oct 27, 2009, 2:36:55 AM (16 years ago)

Author:

Dmitry A. Kuminov

Message:

gui: xsystray: Further coding (established communication channel between the clients and the server, some bug fixes).

Location:

trunk/src/3rdparty/os2/xsystray

Files:

• xsystray.c (modified) (13 diffs)
• xsystray.h (modified) (3 diffs)
• xsystray_api.c (modified) (14 diffs)
• xsystray_api.h (modified) (1 diff)

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

@@ -40 +40 @@
 #include <stdio.h>
 #include <string.h>
+
+
 
 // generic headers
@@ -73 +75 @@
 #pragma hdrstop                     // VAC++ keeps crashing otherwise
 
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
 /* ******************************************************************
  *
@@ -79 +106 @@
  ********************************************************************/
 
-// None currently.
+/*
+ *@@ ICONDATA:
+ *      Per-icon data.
+ */
+
+typedef struct _ICONDATA
+{
+    HWND        hwnd;
+                // associated window
+    ULONG       ulId;
+                // icon ID
+    HPOINTER    hIcon;
+                // icon handle
+    ULONG       ulMsgId;
+                // message ID for notifications
+    PSZ         pszToolTip;
+                // icon tooltip (NULL if none)
+
+} ICONDATA, *PICONDATA;
+
+/*
+ *@@ SYSTRAYDATA:
+ *      Global system tray data.
+ */
+
+typedef struct
+{
+    PICONDATA   pIcons;
+                // array of icons currently shown in the system tray
+                // (left to right)
+    size_t      cIcons;
+                // number of icons in the pIcons array
+    size_t      cIconsMax;
+                // maximum number of icons pIcons can fit
+
+} SYSTRAYDATA, *PSYSTRAYDATA;
+
+ULONG QWL_USER_SERVER_DATA = 0;
 
 /* ******************************************************************
@@ -196 +260 @@
  */
 
+
 BOOL WgtControl(PXCENTERWIDGET pWidget,
                 MPARAM mp1,
@@ -240 +305 @@
  */
 
+
 VOID WgtPaint(HWND hwnd,
               PXCENTERWIDGET pWidget)
@@ -274 +340 @@
 /*
  *@@ fnwpXSysTray:
- *      window procedure for the winlist widget class.
+ *      window procedure for the  widget class.
  *
  *      There are a few rules which widget window procs
@@ -311 +377 @@
 
         case WM_CREATE:
+
+
+
+
+
             WinSetWindowPtr(hwnd, QWL_USER, mp1);
             if (    (!(pWidget = (PXCENTERWIDGET)mp1))
@@ -316 +387 @@
                )
                 // shouldn't happen... stop window creation!!
-                mrc = (MPARAM)TRUE;
+                break;
+
+            pSysTrayData = malloc(sizeof(*pSysTrayData));
+            if (pSysTrayData == NULL)
+                break;
+
+            // initialize the SYSTRAYDATA structure
+            pSysTrayData->cIconsMax = 4;
+            pSysTrayData->pIcons = malloc(sizeof(*pSysTrayData->pIcons) *
+                                          pSysTrayData->cIconsMax);
+            if (pSysTrayData->pIcons == NULL)
+            {
+                free(pSysTrayData);
+                break;
+            }
+            pSysTrayData->cIcons = 0;
+            pWidget->pUser = pSysTrayData;
+
+            // create the "server" window (note that we pass the XCENTERWIDGET
+            // pointer on to it)
+            HWND hwndServer =
+                WinCreateWindow(HWND_DESKTOP, WNDCLASS_WIDGET_XSYSTRAY_SERVER,
+                                NULL, WS_MINIMIZED,
+                                0, 0, 0, 0,
+                                HWND_DESKTOP, HWND_BOTTOM,
+                                0, mp1, NULL);
+            if (hwndServer == NULLHANDLE)
+                break;
+
+            mrc = FALSE; // confirm success
+        }
         break;
 
@@ -353 +454 @@
 
         case WM_DESTROY:
-            // If we had any user data allocated in WM_CREATE
-            // or elsewhere, we'd clean this up here.
+        {
+            // free all system tray data
+            PSYSTRAYDATA pSysTrayData = (PSYSTRAYDATA)pWidget->pUser;
+            size_t i;
+            for (i = 0; i < pSysTrayData->cIcons; ++i)
+            {
+                if (pSysTrayData->pIcons[i].pszToolTip)
+                    free(pSysTrayData->pIcons[i].pszToolTip);
+            }
+            free(pSysTrayData->pIcons);
+            free(pSysTrayData);
+            pWidget->pUser = NULL;
             // We _MUST_ pass this on, or the default widget proc
             // cannot clean up.
             mrc = pWidget->pfnwpDefWidgetProc(hwnd, msg, mp1, mp2);
+
         break;
 
@@ -363 +475 @@
             mrc = pWidget->pfnwpDefWidgetProc(hwnd, msg, mp1, mp2);
 
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
     } // end switch(msg)
 
@@ -428 +625 @@
                              PSZ pszErrorMsg)       // if 0 is returned, 500 bytes of error msg
 {
-    ULONG   ulrc = 0;
-
-    // register our PM window class
-    if (!WinRegisterClass(hab,
-                          WNDCLASS_WIDGET_XSYSTRAY,
-                          fnwpXSysTray,
-                          CS_PARENTCLIP | CS_SIZEREDRAW | CS_SYNCPAINT,
-                          sizeof(PVOID))
+    ULONG       ulrc = 0;
+    CLASSINFO   ClassInfo;
+
+    LOGF(("hmodPlugin %x\n", hmodPlugin));
+
+    do
+    {
+        // register our PM window class
+        if (!WinRegisterClass(hab,
+                             WNDCLASS_WIDGET_XSYSTRAY,
+                             fnwpXSysTray,
+                             CS_PARENTCLIP | CS_SIZEREDRAW | CS_SYNCPAINT,
+                             sizeof(PVOID))
                                 // extra memory to reserve for QWL_USER
-                         )
-        // error registering class: report error then
-        strcpy(pszErrorMsg, "WinRegisterClass failed.");
-    else
-    {
+            )
+        {
+            LOG(("WinRegisterClass(%s) failed with %lX",
+                 WNDCLASS_WIDGET_XSYSTRAY, WinGetLastError(hab)));
+            break;
+        }
+
+        // get the window data size for the WC_FRAME class (any window class
+        // that specifies CS_FRAME must have at least this number, otherise
+        // WinRegisterClass returns 0x1003
+        if (!WinQueryClassInfo(hab, (PSZ)WC_FRAME, &ClassInfo))
+            break;
+        QWL_USER_SERVER_DATA = ClassInfo.cbWindowData;
+
+        if (!WinRegisterClass(hab,
+                              WNDCLASS_WIDGET_XSYSTRAY_SERVER,
+                              fnwpXSysTrayServer,
+                              CS_FRAME,
+                              QWL_USER_SERVER_DATA + sizeof(PVOID))
+                                // extra memory to reserve for QWL_USER
+            )
+        {
+            // error registering class: report error then
+            snprintf(pszErrorMsg, 500, "WinRegisterClass(%s) failed with %lX",
+                     WNDCLASS_WIDGET_XSYSTRAY_SERVER, WinGetLastError(hab));
+            break;
+        }
+
         // no error:
         // return widget classes array
@@ -449 +674 @@
         ulrc = sizeof(G_WidgetClasses) / sizeof(G_WidgetClasses[0]);
     }
+
+
+
+
 
     return ulrc;
@@ -489 +718 @@
 {
     *pulMajor = 0;
-    *pulMinor = 1;
-    *pulRevision = 0;
-}
-
+    *pulMinor = ;
+    *pulRevision = ;
+}
+

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

@@ -16 +16 @@
 #ifndef XSYSTRAY_HEADER_INCLUDED
 #define XSYSTRAY_HEADER_INCLUDED
+
+
+
+
 
 #define WNDCLASS_WIDGET_XSYSTRAY_SERVER "XWPCenterExtendedSysTrayServer"
@@ -35 +39 @@
 } SYSTRAYCMD;
 
-// NOTE: the idea is to fit SYSTRAYMSGDATA into a page (4K) since this is the
+// NOTE: the idea is to fit SYSTRAYDATA into a page (4K) since this is the
 // smallest piece DosAllocSharedMem() allocates anyway.
 typedef struct
@@ -73 +77 @@
             // set to true by the recipient if it processes the message
 
-} SYSTRAYMSGDATA, *PSYSTRAYMSGDATA;
+} SYSTRAYDATA;
 
 #endif // XSYSTRAY_HEADER_INCLUDED

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

@@ -27 +27 @@
 
 static HWND G_hwndSysTray = NULLHANDLE;
-static int G_itlsSysTrayMsgData = -1;
+static int G_itlsSysTrayData = -1;
 
 static HWND FindSysTrayWindow()
@@ -46 +46 @@
 }
 
-static BOOL SendSysTrayCtlMsg(PSYSTRAYMSGDATA pData)
+static BOOL SendSysTrayCtlMsg(PSYSTRAYDATA pData)
 {
     APIRET arc;
@@ -70 +70 @@
             arc = ERROR_INVALID_HANDLE;
             if (WinQueryWindowProcess(G_hwndSysTray, &pid, &tid))
-                arc = DosGiveSharedMem(__libc_TLSGet(G_itlsSysTrayMsgData),
+                arc = DosGiveSharedMem(__libc_TLSGet(G_itlsSysTrayData),
                                        pid, PAG_READ | PAG_WRITE);
             if (arc != NO_ERROR)
@@ -80 +80 @@
         mrc = WinSendMsg(G_hwndSysTray, WM_XST_CONTROL, pData, NULL);
         if (mrc == (MRESULT)TRUE && pData->bAcknowledged)
-            break;
+            ;
 
         // if we failed to send the message, it may mean that XCenter was restarted
@@ -98 +98 @@
 }
 
-static PSYSTRAYMSGDATA GetSysTrayMsgDataPtr()
+static PSYSTRAYDataPtr()
 {
     APIRET arc;
 
     // allocate a thread local storage entry if not done so
-    if (G_itlsSysTrayMsgData == -1)
+    if (G_itlsSysTrayData == -1)
     {
         // @todo does XWorkplace have its own TLS? Not? Use
@@ -109 +109 @@
         // to the lack of space in that area)
         int itls = __libc_TLSAlloc();
-        if (!__atomic_cmpxchg32(&G_itlsSysTrayMsgData, itls, -1))
+        if (!__atomic_cmpxchg32(&G_itlsSysTrayData, itls, -1))
         {
             // another thread has already got an entry, discard our try
@@ -116 +116 @@
         }
 
-        if (G_itlsSysTrayMsgData == -1)
+        if (G_itlsSysTrayData == -1)
             return NULL;
     }
 
-    // allocate a SYSTRAYMSGDATA struct for this thread if not done so
-    PSYSTRAYMSGDATA pData = __libc_TLSGet(G_itlsSysTrayMsgData);
-    if (!pData)
-    {
-        arc = DosAllocSharedMem((PVOID)&pData, NULL, sizeof(SYSTRAYMSGDATA),
+    // allocate a SYSTRAYDATA struct for this thread if not done so
+    PSYSTRAYData);
+    if (!pData)
+    {
+        arc = DosAllocSharedMem((PVOID)&pData, NULL, sizeof(SYSTRAYDATA),
                                 PAG_COMMIT | PAG_READ | PAG_WRITE | OBJ_GIVEABLE);
         if (arc != NO_ERROR)
             return NULL;
 
-        __libc_TLSSet(G_itlsSysTrayMsgData, pData);
+        __libc_TLSSet(G_itlsSysTrayData, pData);
 
         // note that we don't ever free the allocated block since our API doesn't
@@ -159 +159 @@
                             PULONG pulRevision) // out: revision number
 {
-    PSYSTRAYMSGDATA pData = GetSysTrayMsgDataPtr();
+    PSYSTRAYDataPtr();
     if (!pData)
         return FALSE;
@@ -166 +166 @@
     pData->hwndSender = NULLHANDLE;
 
-    BOOL rc = SendSysTrayCtlMsg(pData);
-    if (rc)
+    BOOL rc = SendSysTrayCtlMsg(pData);
+    if (rc)
     {
         if (pulMajor)
@@ -177 +177 @@
     }
 
-    return rc;
+    return rc;
 }
 
@@ -215 +215 @@
                        ULONG ulFlags)   // in: flags (not currently used, must be 0)
 {
-    PSYSTRAYMSGDATA pData = GetSysTrayMsgDataPtr();
+    PSYSTRAYDataPtr();
     if (!pData)
         return FALSE;
@@ -240 +240 @@
                           ULONG ulId)   // in: icon ID to remove
 {
-    PSYSTRAYMSGDATA pData = GetSysTrayMsgDataPtr();
+    PSYSTRAYDataPtr();
     if (!pData)
         return FALSE;
@@ -272 +272 @@
                               PSZ pszText)  // in: tooltip text
 {
-    PSYSTRAYMSGDATA pData = GetSysTrayMsgDataPtr();
+    PSYSTRAYDataPtr();
     if (!pData)
         return FALSE;
@@ -361 +361 @@
 ULONG xstGetSysTrayMaxTextLen()
 {
-    return sizeof(((PSYSTRAYMSGDATA)0)->u.tooltip.szText);
-}
-
+    return sizeof(((PSYSTRAYDATA)0)->u.tooltip.szText);
+}
+

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

@@ -21 +21 @@
 #endif
 
-// notification code constants for WM_XST_NOTIFY, refer to xstAddSysTrayIcon()
+// notification code constants for the notification messages sent by the system
+// tray (refer to xstAddSysTrayIcon() for details)
 #define XST_IN_MOUSE    0x0001