Changeset 131 for trunk/src/user32/user32.cpp

Timestamp:

Jun 20, 1999, 4:02:13 PM (26 years ago)

Author:

sandervl

Message:

Comctl32 crash on exit fix + user32 bugfix for winhlp32 changed to be similar to Wine

File:

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

trunk/src/user32/user32.cpp

@@ -1 @@
-/* $Id: user32.cpp,v 1.7 1999-06-12 20:44:07 cbratschi Exp $ */
+/* $Id: user32.cpp,v 1.8 1999-06-20 14:02:13 sandervl Exp $ */
 
 /*
@@ -545 +545 @@
                                  int       nWidth,
                                  int       nHeight,
-                                 HWND      arg9,
+                                 HWND      parent,
                                  HMENU     arg10,
                                  HINSTANCE arg11,
@@ -557 +557 @@
         arg3 = (LPCSTR)"CRASH, CRASH";
 
-//SvL: This break generic.exe & notepad.exe (probably many others too)
-//These parameters can be CW_USEDEFAULT, which is 0x80000000
-#if 0
-  if (nWidth < 0) nWidth  = 0;
-  if (x      < 0) x       = 0;
-  if (nHeight< 0) nHeight = 0;
-  if (y      < 0) y       = 0;
-#endif
-
-// 6-12-99 CB: WS_CLIPCHILDREN not set -> controls not redrawn
-// Problems with group boxes
-
-#ifndef WS_CLIPCHILDREN
-  #define WS_CLIPCHILDREN            0x20000000L
-#endif
-
-  dwStyle |= WS_CLIPCHILDREN;
-
-//SvL: Breaks applications
-#if 0
-  /* @@@PH 98/06/21 redraw problems disappear when WS_SYNCPAINT is on */
-#ifndef WS_VISIBLE
-  #define WS_VISIBLE                 0x80000000L
-#endif
-
-#ifndef WS_SYNCPAINT
-  #define WS_SYNCPAINT               0x02000000L
-#endif
-
-#ifndef WS_CLIPSYBLINGS
-  #define WS_CLIPSYBLINGS            0x10000000L
-#endif
-
-  dwStyle |= WS_SYNCPAINT;
-
-  /* @@@PH 98/06/21 experimental fix for WinHlp32 */
-  // SOL.EXE crashes here, but WINHLP32 does not display the
-  // navigation buttons otherwise.
-  dwStyle |= WS_VISIBLE;
-#endif
+  // 6-12-99 CB: WS_CLIPCHILDREN not set -> controls not redrawn
+  // Problems with group boxes
+
+  //SvL: Correct window style (like Wine does) instead
+  if(dwStyle & WS_CHILD) {
+        dwStyle |= WS_CLIPSIBLINGS;
+        if(!(dwStyle & WS_POPUP)) {
+                dwStyle |= WS_CAPTION;
+        }
+  }
+  if(dwExStyle & WS_EX_DLGMODALFRAME)
+  {
+        dwStyle &= ~WS_THICKFRAME;
+  }
 
 #ifdef DEBUG
     WriteLog("USER32:  CreateWindow: dwExStyle = %X\n", dwExStyle);
     if((int)arg2 >> 16 != 0)
-     WriteLog("USER32:  CreateWindow: classname = %s\n", arg2);
+         WriteLog("USER32:  CreateWindow: classname = %s\n", arg2);
     else WriteLog("USER32:  CreateWindow: classname = %X\n", arg2);
     WriteLog("USER32:  CreateWindow: windowname= %s\n", arg3);
@@ -609 +583 @@
     WriteLog("USER32:  CreateWindow: nWidth    = %d\n", nWidth);
     WriteLog("USER32:  CreateWindow: nHeight   = %d\n", nHeight);
-    WriteLog("USER32:  CreateWindow: parent    = %X\n", arg9);
+    WriteLog("USER32:  CreateWindow: parent    = %X\n", parent);
     WriteLog("USER32:  CreateWindow: hwmenu    = %X\n", arg10);
     WriteLog("USER32:  CreateWindow: hinstance = %X\n", arg11);
@@ -640 +614 @@
                           nWidth,
                           nHeight,
-                          arg9,
+                          parent,
                           arg10,
                           arg11,
@@ -653 +627 @@
         window->SetWindowHandle(hwnd);
     }
+    //SvL: Taken from Wine
+    if(dwStyle & WS_CHILD && !(dwExStyle & WS_EX_NOPARENTNOTIFY) )
+    {
+        /* Notify the parent window only */
+        SendMessageA(parent, WM_PARENTNOTIFY, MAKEWPARAM(WM_CREATE, 0), (LPARAM)hwnd );
+    }
+
     dprintf(("USER32:  ************CreateWindowExA %s (%d,%d,%d,%d), hwnd = %X\n", arg2, x, y, nWidth, nHeight, hwnd));
     return(hwnd);