Re: (try 6)[1/5] imm32: Move thread data from TLSEntry to an internal list

[Alexandre Julliard]

Aric Stewart  writes:
> @@ -1512,11 +1589,15 @@ BOOL WINAPI ImmGetConversionStatus(
>   */
>  HWND WINAPI ImmGetDefaultIMEWnd(HWND hWnd)
>  {
> - if (IMM_GetThreadData()->hwndDefault == NULL)
> -
>    IMM_GetThreadData()->hwndDefault = CreateWindowExW( WS_EX_TOOLWINDOW,
> - HWND ret;
> - IMMThreadData* thread_data = IMM_GetThreadData(0);
> - if (thread_data->hwndDefault == NULL)
> -
>    thread_data->hwndDefault = CreateWindowExW( WS_EX_TOOLWINDOW,
>                szwIME, NULL, WS_POPUP, 0, 0, 1, 1, 0, 0, 0, 0);
> - TRACE("Default is %p\n",IMM_GetThreadData()->hwndDefault);
> - return IMM_GetThreadData()->hwndDefault;
> - ret = thread_data->hwndDefault;
> - LeaveCriticalSection(&threaddata_cs);
> - TRACE("Default is %p\n",ret);
> - return ret;

You can't create a window while holding the critical section, this will
cause deadlocks.

(try 6)[1/5] imm32: Move thread data from TLSEntry to an internal list

[Aric Stewart]

On 2/25/15 1:57 AM, Alexandre Julliard wrote:
> Aric Stewart  writes:
>
> You can't create a window while holding the critical section, this will
> cause deadlocks.

Thanks, fixed and resent

-aric