Re: [PATCH v10 0/8] MR729: windows.media.speech: Initial recognition session work.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/windows.media.speech/recognizer.c:
>  return ref;
>
> }
>
> +static HRESULT session_join_worker_thread( ISpeechContinuousRecognitionSession *iface )
> +{
> - struct session *impl = impl_from_ISpeechContinuousRecognitionSession(iface);
> -
> - WakeConditionVariable(&impl->cv);

So I missed that in the previous comments and it means things are probably fine then, but I'd find it more obvious if the wake up is done right after the flags are changed, rather than doing it here without any apparent reason (it's not because you wake it up that the thread will exit).