suggestions about the throw feature of the mouse

[x1244]

In the Trackball of VSG, the throw feature of the mouse is supported. This feature is achieved through the statement:

if (supportsThrow) _thrown = _previousPointerEvent && (buttonRelease.time == _previousPointerEvent->time);

However, the condition that checks if buttonRelease.time is equal to _previousPointerEvent->time is too stringent, or in a practical sense, these two times will not be exactly equal. It would be more appropriate to check if the time difference between them is within a millisecond range, as shown in the following code:

if (supportsThrow) _thrown = _previousPointerEvent && 
(std::chrono::duration_cast<std::chrono::milliseconds>
 (buttonRelease.time - _previousPointerEvent->time).count() == 0);

Would you accept such a modification?

[robertosfield]

This is something I've wanted to look at. Xcb/X11 generates mouse button release and move events with the time timestamp, but Windows doesn't provide this level of info so will need some other means to confirm that the even release is happening while the mouse is still moving.

A customizable time delta is a potential way to solve it, so if you can get it work reliably with submit a PR :-)

[x1244]

I have already submitted a PR. I believe there is no need to customize the time difference value between mouse movement and release events. Detecting whether these two events occur simultaneously within a 1-millisecond interval should suffice for the vast majority of scenarios.