USER: creating 16bit windows convert null instances to current process (try #3, should work now)

[Joseph Garvin]

It's been a about a month since I sent this in and it still hasn't been 
applied. Is there something wrong with the patch?

Joseph Garvin wrote:

> Fixed up the patch according to Dmitry's suggestions :)
>
> Changelog
>   Fixed instance = 0 not converted to the current process when making a
> window. Broke menus in some apps. (Thanks to Mike)
>
>
>------------------------------------------------------------------------
>
>--- dlls/user/win.c.1	2005-12-31 03:03:19.000000000 -0700
>+++ dlls/user/win.c	2006-01-01 01:02:43.000000000 -0700
>@@ -1145,6 +1145,18 @@ HWND16 WINAPI CreateWindowEx16( DWORD ex
>     cs.cx = (width == CW_USEDEFAULT16) ? CW_USEDEFAULT : (INT)width;
>     cs.cy = (height == CW_USEDEFAULT16) ? CW_USEDEFAULT : (INT)height;
> 
>+    /* Correct if instance is null now, because later we won't
>+       be able to tell the difference between a 16-bit NULL and
>+       a 32-bit NULL */
>+
>+    if (!instance) 
>+    {
>+        HTASK16 hTask = GetCurrentTask();
>+        TDB *tdb = GlobalLock16(hTask);
>+        instance = tdb->hInstance;
>+        GlobalUnlock16(hTask);
>+    }
>+
>     /* Create the window */
> 
>     cs.lpCreateParams = data;
>  
>
>------------------------------------------------------------------------
>
>
>  
>