Re: [PATCH v5 0/12] MR1948: Draft: windows.media.speech: Recognition session work the second.

[Jinoh Kang (＠iamahuman)]

Jinoh Kang (@iamahuman) commented about dlls/windows.media.speech/recognizer.c:
> +        thread = impl->worker_thread;
> +        impl->worker_thread = INVALID_HANDLE_VALUE;
> +        impl->worker_running = FALSE;
> +        impl->worker_paused = FALSE;
> +        impl->recognizer_state = SpeechRecognizerState_Idle;
> +    }
> +    else
> +        invalid_state = TRUE;
> +    LeaveCriticalSection(&impl->cs);
> +
> +    if (invalid_state && SUCCEEDED(hr))
> +        hr = COR_E_INVALIDOPERATION;
> +    else if (!invalid_state)
> +    {
> +        SetEvent(impl->worker_control_event);
> +        session_join_worker_thread(thread);
        WaitForSingleObject(thread, INFINITE);
        CloseHandle(thread);

Readers cannot infer that `session_join_worker_thread` closes the given handle from the function's name.  Instead of renaming it to `session_join_close_worker_thread`, might as well open code it.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/1948#note_21047