[PATCH v4 7/7] winex11.drv: Don't react to small slow mouse movements.
Derek Lesho
dereklesho52 at gmail.com
Tue Jul 16 19:00:20 CDT 2019
This is a modified version of staging's mouse-movements patch that applies
cleanly on the first 6 patches I sent.
On Tue, Jul 16, 2019 at 7:57 PM Derek Lesho <dereklesho52 at gmail.com> wrote:
> Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=42631
> From: Jordan Galby <gravemind2a+wine at gmail.com>
> Signed-off-by: Derek Lesho <dereklesho52 at Gmail.com>
> ---
> dlls/winex11.drv/mouse.c | 66 +++++++++++++++++++++++++++++----------
> dlls/winex11.drv/x11drv.h | 2 ++
> 2 files changed, 51 insertions(+), 17 deletions(-)
>
> diff --git a/dlls/winex11.drv/mouse.c b/dlls/winex11.drv/mouse.c
> index 1ceac3d7f2..e9cb591cb5 100644
> --- a/dlls/winex11.drv/mouse.c
> +++ b/dlls/winex11.drv/mouse.c
> @@ -257,6 +257,10 @@ static void update_relative_valuators(XIAnyClassInfo
> **valuators, int n_valuator
>
> thread_data->x_rel_valuator.number = -1;
> thread_data->y_rel_valuator.number = -1;
> + thread_data->x_rel_valuator.accum = 0;
> + thread_data->y_rel_valuator.accum = 0;
> + thread_data->x_rel_valuator.raw_accum = 0;
> + thread_data->y_rel_valuator.raw_accum = 0;
>
> for (i = 0; i < n_valuators; i++)
> {
> @@ -1796,15 +1800,11 @@ static BOOL X11DRV_RawMotion( XGenericEventCookie
> *xev )
> input.u.mi.dwFlags = MOUSEEVENTF_MOVE;
> input.u.mi.time = EVENT_x11_time_to_win32_time( event->time );
> input.u.mi.dwExtraInfo = 0;
> - input.u.mi.dx = 0;
> - input.u.mi.dy = 0;
>
> raw_input.header.dwType = RIM_TYPEMOUSE;
> raw_input.data.mouse.u.usButtonFlags = 0;
> raw_input.data.mouse.u.usButtonData = 0;
> raw_input.data.mouse.ulExtraInformation = 0;
> - raw_input.data.mouse.lLastX = 0;
> - raw_input.data.mouse.lLastY = 0;
>
> virtual_rect = get_virtual_screen_rect();
>
> @@ -1815,40 +1815,72 @@ static BOOL X11DRV_RawMotion( XGenericEventCookie
> *xev )
> raw_val = *raw_values++;
> if (i == x_rel->number)
> {
> - input.u.mi.dx = dx = val;
> + dx = val;
> if (x_rel->min < x_rel->max)
> - input.u.mi.dx = val * (virtual_rect.right -
> virtual_rect.left)
> + dx = val * (virtual_rect.right - virtual_rect.left)
> / (x_rel->max - x_rel->min);
>
> - raw_input.data.mouse.lLastX = raw_dx = raw_val;
> + raw_dx = raw_val;
> }
> if (i == y_rel->number)
> {
> - input.u.mi.dy = dy = val;
> + dy = val;
> if (y_rel->min < y_rel->max)
> - input.u.mi.dy = val * (virtual_rect.bottom -
> virtual_rect.top)
> + dy = val * (virtual_rect.bottom - virtual_rect.top)
> / (y_rel->max - y_rel->min);
>
> - raw_input.data.mouse.lLastY = raw_dy = raw_val;
> + raw_dy = raw_val;
> }
> }
>
> if (broken_rawevents && is_old_motion_event( xev->serial ))
> {
> - TRACE( "pos %d,%d old serial %lu, ignoring\n", input.u.mi.dx,
> input.u.mi.dy, xev->serial );
> + TRACE( "pos %d,%d old serial %lu, ignoring\n", (LONG) dx, (LONG)
> dy, xev->serial );
> return FALSE;
> }
>
> - if (thread_data->xi2_state == xi_extra)
> + /* Accumulate the *double* motions so sub-pixel motions
> + * wont be lost when sent/cast to *LONG* target fields.
> + */
> +
> + x_rel->accum += dx;
> + y_rel->accum += dy;
> + if (fabs(x_rel->accum) < 1.0 && fabs(y_rel->accum) < 1.0)
> {
> - TRACE( "pos %d,%d (event %f,%f)\n", input.u.mi.dx, input.u.mi.dy,
> dx, dy );
> - __wine_send_input( 0, &input );
> + TRACE( "accumulating raw motion (event %f,%f, accum %f,%f)\n",
> dx, dy, x_rel->accum, y_rel->accum );
> }
> + else
> + {
> + input.u.mi.dx = x_rel->accum;
> + input.u.mi.dy = y_rel->accum;
> + x_rel->accum -= input.u.mi.dx;
> + y_rel->accum -= input.u.mi.dy;
>
> - if ( LIST_ENTRY(list_head(&g_x11_threads), struct x11drv_thread_data,
> entry) == thread_data)
> + if (thread_data->xi2_state == xi_extra)
> + {
> + TRACE( "pos %d,%d (event %f,%f)\n", input.u.mi.dx,
> input.u.mi.dy, dx, dy );
> + __wine_send_input( 0, &input );
> + }
> + }
> +
> + x_rel->raw_accum += raw_dx;
> + y_rel->raw_accum += raw_dy;
> + if (fabs(x_rel->raw_accum) < 1.0 && fabs(y_rel->raw_accum) < 1.0)
> + {
> + TRACE( "accumulating raw motion (event %f,%f, accum %f,%f)\n",
> raw_dx, raw_dy, x_rel->raw_accum, y_rel->raw_accum );
> + }
> + else
> {
> - TRACE("raw event %f,%f\n", raw_dx, raw_dy);
> - __wine_send_raw_input( &raw_input );
> + raw_input.data.mouse.lLastX = x_rel->raw_accum;
> + raw_input.data.mouse.lLastY = y_rel->raw_accum;
> + x_rel->raw_accum -= raw_input.data.mouse.lLastX;
> + y_rel->raw_accum -= raw_input.data.mouse.lLastY;
> +
> + if ( LIST_ENTRY(list_head(&g_x11_threads), struct
> x11drv_thread_data, entry) == thread_data)
> + {
> + TRACE("raw event %d,%d(event %f,%f)\n",
> raw_input.data.mouse.lLastX, raw_input.data.mouse.lLastY, raw_dx, raw_dy);
> + __wine_send_raw_input( &raw_input );
> + }
> }
>
> return TRUE;
> diff --git a/dlls/winex11.drv/x11drv.h b/dlls/winex11.drv/x11drv.h
> index 378c1d7508..06b09730ee 100644
> --- a/dlls/winex11.drv/x11drv.h
> +++ b/dlls/winex11.drv/x11drv.h
> @@ -320,6 +320,8 @@ struct x11drv_valuator_data
> double min;
> double max;
> int number;
> + double accum;
> + double raw_accum;
> };
>
> struct x11drv_thread_data
> --
> 2.22.0
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20190716/9e48edff/attachment.html>
More information about the wine-devel
mailing list