Changeset 2312 for trunk/src/user32/new/oslibmsg.cpp

Timestamp:

Jan 3, 2000, 10:37:17 PM (26 years ago)

Author:

sandervl

Message:

PostThreadMessage bugfix

File:

• trunk/src/user32/new/oslibmsg.cpp (modified) (6 diffs)

trunk/src/user32/new/oslibmsg.cpp

@@ -1 @@
-/* $Id: oslibmsg.cpp,v 1.6 2000-01-01 14:57:18 cbratschi Exp $ */
+/* $Id: oslibmsg.cpp,v 1.7 2000-01-03 21:37:17 sandervl Exp $ */
 /*
  * Window message translation functions for OS/2
@@ -178 +178 @@
             thdb->msgstate++; //odd -> next call to our PM window handler should dispatch the translated msg
             memcpy(&thdb->msg, msg, sizeof(MSG));
-        }
-        return (LONG)WinDispatchMsg(thdb->hab, &os2msg);
+            return (LONG)WinDispatchMsg(thdb->hab, &os2msg);
+        }
+        //SvL: TODO; What to do if messages posted by PostThreadMessage are
+        //     dispatched? Wine doesn't appear to do this.
+        //     If we call WinDispatchMsg, every window created by the thread
+        //     receives this message. Does this also happen in NT?
+        return 0;
   }
   else {//is this allowed?
@@ -193 +198 @@
  BOOL rc, eaten;
  THDB *thdb;
+ QMSG  os2msg;
 
   thdb = GetThreadTHDB();
@@ -223 +229 @@
         }
         while(rc == FALSE);
+
+        return rc;
   }
   else
@@ -228 +236 @@
     do {
         eaten = FALSE;
-        rc = WinGetMsg(thdb->hab, MsgThreadPtr, TranslateWinMsg(uMsgFilterMin, TRUE), TranslateWinMsg(uMsgFilterMax, FALSE), 0);
-        if (MsgThreadPtr->msg == WM_TIMER)
-            eaten = TIMER_HandleTimer (MsgThreadPtr);
+        rc = WinGetMsg(thdb->hab, &os2msg, TranslateWinMsg(uMsgFilterMin, TRUE), TranslateWinMsg(uMsgFilterMax, FALSE), 0);
+        if (os2msg.msg == WM_TIMER)
+            eaten = TIMER_HandleTimer(&os2msg);
     } while (eaten);
   }
-  OS2ToWinMsgTranslate((PVOID)thdb, MsgThreadPtr, pMsg, isUnicode, MSG_REMOVE);
+  if(rc) {
+        OS2ToWinMsgTranslate((PVOID)thdb, &os2msg, pMsg, isUnicode, MSG_REMOVE);
+        memcpy(MsgThreadPtr, &os2msg, sizeof(QMSG));
+  }
   return rc;
 }
@@ -393 +404 @@
 }
 //******************************************************************************
+BOOL    _System _O32_PostThreadMessage( DWORD, UINT, WPARAM, LPARAM );
+
+inline BOOL O32_PostThreadMessage(DWORD a, UINT b, WPARAM c, LPARAM d)
+{
+ BOOL yyrc;
+ USHORT sel = RestoreOS2FS();
+
+    yyrc = _O32_PostThreadMessage(a, b, c, d);
+    SetFS(sel);
+
+    return yyrc;
+} 
 //******************************************************************************
 BOOL OSLibPostThreadMessage(ULONG threadid, UINT msg, WPARAM wParam, LPARAM lParam, BOOL fUnicode)
 {
- THDB *thdb = GetTHDBFromThreadId(threadid);
+// THDB *thdb = GetTHDBFromThreadId(threadid);
  POSTMSG_PACKET *packet = (POSTMSG_PACKET *)_smalloc(sizeof(POSTMSG_PACKET));
 
-    if(thdb == NULL) {
-        dprintf(("OSLibPostThreadMessage: thread %x not found!", threadid));
-        return FALSE;
-    }
+//    if(thdb == NULL) {
+//        dprintf(("OSLibPostThreadMessage: thread %x not found!", threadid));
+//        return FALSE;
+//    }
     dprintf(("PostThreadMessageA %x %x %x %x", threadid, msg, wParam, lParam));
     packet->Msg = msg;
     packet->wParam = wParam;
     packet->lParam = lParam;
-    return WinPostQueueMsg(thdb->hmq, WIN32APP_POSTMSG, (MPARAM)((fUnicode) ? WIN32MSG_MAGICW : WIN32MSG_MAGICA), (MPARAM)packet);
-}
-//******************************************************************************
-//******************************************************************************
-
+    return _O32_PostThreadMessage(threadid, WIN32APP_POSTMSG-OPEN32_MSGDIFF, ((fUnicode) ? WIN32MSG_MAGICW : WIN32MSG_MAGICA), (LPARAM)packet);
+}
+//******************************************************************************
+//******************************************************************************
+