Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@Krzmbrzl
Copy link
Member

Backport

This will backport the following commits from master to 1.4.x:

Questions ?

Please refer to the Backport tool documentation

Prior to the implementaiton of raw mouse input, a low-level mouse hook was used for capturing mouse events. Windows would emit a single WM_* event per button state change. With the move to capturing raw mouse input events, it was expected that this behavior remained, despite now needing to listen for WM_INPUT events.

However, unlike when using the low-level hook, WM_INPUT events can contain data for multiple simultaneous mouse state changes. If this occurred, Mumble would register none of the state changes.

Changing the logic to account for multiple state changes in a single event correctly allows for all states changes to be registered.

Fixes mumble-voip#5706

(cherry picked from commit 01af7d9)
@Krzmbrzl Krzmbrzl merged commit dadfa50 into mumble-voip:1.4.x Aug 11, 2022
@Krzmbrzl Krzmbrzl deleted the backport/1.4.x/pr-5739 branch November 9, 2022 18:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants