Changeset 3702 for trunk/src/user32/windowclass.cpp

Timestamp:

Jun 13, 2000, 11:26:30 PM (25 years ago)

Author:

sandervl

Message:

when registering a class, check if it already exists and return error

File:

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

trunk/src/user32/windowclass.cpp

@@ -1 @@
-/* $Id: windowclass.cpp,v 1.10 2000-05-28 16:43:48 sandervl Exp $ */
+/* $Id: windowclass.cpp,v 1.11 2000-06-13 21:26:30 sandervl Exp $ */
 /*
  * Win32 Window Class Code for OS/2
@@ -45 +45 @@
  Win32WndClass *wclass;
 
-    dprintf(("RegisterClassA\n"));
+   dprintf(("RegisterClassA\n"));
    //CB: size new in ex structure
    wc.cbSize = sizeof(wc);
@@ -51 +51 @@
    wc.hIconSm = 0;
 
+   if(Win32WndClass::FindClass(wc.hInstance, (LPSTR)wc.lpszClassName)) {
+        dprintf(("RegisterClassA %x %x already exists", wc.hInstance, wc.lpszClassName));
+        SetLastError(ERROR_CLASS_ALREADY_EXISTS);
+        return 0;
+   }
+
    wclass = new Win32WndClass(&wc,FALSE);
    if(wclass == NULL) {
-        dprintf(("RegisterClassA wclass == NULL!"));
+        dprintf(("ERROR: RegisterClassA winclass == NULL!"));
+        DebugInt3();
         return(0);
    }
@@ -63 +70 @@
 {
  Win32WndClass *wclass;
+
+   if(Win32WndClass::FindClass(lpWndClass->hInstance, (LPSTR)lpWndClass->lpszClassName)) {
+        dprintf(("RegisterClassExA %x %x already exists", lpWndClass->hInstance, lpWndClass->lpszClassName));
+        SetLastError(ERROR_CLASS_ALREADY_EXISTS);
+        return 0;
+   }
 
    dprintf(("RegisterClassExA\n"));
    wclass = new Win32WndClass((WNDCLASSEXA *)lpWndClass,FALSE);
    if(wclass == NULL) {
-        dprintf(("RegisterClassExA wclass == NULL!"));
+        dprintf(("ERROR: RegisterClassExA winclass == NULL!"));
+        DebugInt3();
         return(0);
    }
@@ -77 +91 @@
 {
  ATOM rc;
- WNDCLASSEXA wclass;
+ WNDCLASSEXA wc;
  Win32WndClass *winclass;
 
     dprintf(("RegisterClassW\n"));
     //CB: size new in ex structure
-    wclass.cbSize = sizeof(wclass);
-    memcpy(&wclass.style, lpwc, sizeof(WNDCLASSA));
-
-    winclass = new Win32WndClass((WNDCLASSEXA *)&wclass, TRUE);
+    wc.cbSize = sizeof(wc);
+    memcpy(&wc.style, lpwc, sizeof(WNDCLASSA));
+
+    if(Win32WndClass::FindClass(wc.hInstance, (LPWSTR)wc.lpszClassName)) {
+        dprintf(("RegisterClassW %x %x already exists", wc.hInstance, wc.lpszClassName));
+        SetLastError(ERROR_CLASS_ALREADY_EXISTS);
+        return 0;
+    }
+
+    winclass = new Win32WndClass((WNDCLASSEXA *)&wc, TRUE);
     if(winclass == NULL) {
-        dprintf(("RegisterClassW wclass == NULL!"));
-        return(0);
+        dprintf(("ERROR: RegisterClassW winclass == NULL!"));
+        DebugInt3();
+        return 0;
     }
     rc = winclass->getAtom();
@@ -99 +120 @@
 {
  ATOM rc;
- WNDCLASSEXA wclass;
+ WNDCLASSEXA wc;
  Win32WndClass *winclass;
 
     dprintf(("RegisterClassExW\n"));
-    memcpy(&wclass, lpwc, sizeof(WNDCLASSEXA));
-
-    winclass = new Win32WndClass((WNDCLASSEXA *)&wclass, TRUE);
+    memcpy(&wc, lpwc, sizeof(WNDCLASSEXA));
+
+    if(Win32WndClass::FindClass(wc.hInstance, (LPWSTR)wc.lpszClassName)) {
+        dprintf(("RegisterClassW %x %x already exists", wc.hInstance, wc.lpszClassName));
+        SetLastError(ERROR_CLASS_ALREADY_EXISTS);
+        return 0;
+    }
+
+    winclass = new Win32WndClass((WNDCLASSEXA *)&wc, TRUE);
     if(winclass == NULL) {
-        dprintf(("RegisterClassExW wclass == NULL!"));
+        dprintf(("ERROR: RegisterClassExW winclass == NULL!"));
+        DebugInt3();
         return(0);
     }