Re: [PATCH 2/2] dinput: Raise priority of device threads.

[Kai Krakow]

Am Mo., 10. Dez. 2018 um 19:39 Uhr schrieb Alex Henrie :
> On Fri, Dec 7, 2018 at 12:01 PM Kai Krakow  wrote:
> > diff --git a/dlls/dinput/dinput_main.c b/dlls/dinput/dinput_main.c
> > index 6ed764e7485..8b6f393cd45 100644
> > --- a/dlls/dinput/dinput_main.c
> > +++ b/dlls/dinput/dinput_main.c
> > @@ -1815,6 +1815,7 @@ static BOOL check_hook_thread(void)
> >      }
> >      else if (list_empty(&direct_input_list) && hook_thread)
> >      {
> > -
> >    SetThreadPriority(hook_thread, THREAD_PRIORITY_TIME_CRITICAL);
> >    DWORD tid = hook_thread_id;
> >
> >    hook_thread_id = 0;
>
> To avoid compiler warnings, the call to SetThreadPriority needs to go
> after DWORD tid is declared.

Thanks for noticing. I've resent a v2 patch.

BTW: Under staging, this (and the other patch) seems to fix occasional
drops of trigger axis events. At least I no longer have an issue with
"stuck" gamepad triggers. Without staging, it obviously does nothing.

Regards,
Kai