blender
/
blender
136
406
Fork 589
Code Issues 6.3k Pull Requests 765 Projects 19 Wiki Activity

Core: Explicitly throttle mousemove events #110148

Closed
Joseph Eagar wants to merge 1 commits from JosephEagar/blender:fix-inbetween-mousemove into main
pull from: JosephEagar/blender:fix-inbetween-mousemove
merge into: blender:main

When changing the target branch, be careful to rebase the branch in your fork to match. See documentation.
Conversation 3 Commits 1 Files Changed 2 +30 -9

1 Commits

Author SHA1 Message Date
Joseph Eagar 4a6de9c13e Core: Explicitly throttle mousemove events 
The behavior of INBETWEEN_MOUSEMOVE was indeterminate.
Apparently the idea was to drop mousemove events for some
tools during heavy CPU load; this was implemented by
changing the type of runs of MOUSEMOVE events inside the
event queue to INBETWEEN_MOUSEMOVE, with only the event
at the head of the queue MOUSEMOVE.

Basically the size of the event queue was used to throttle
MOUSEMOVE events, presumably under the assumption that
the size of the queue is proportional to CPU and GPU load.
This turned out to be incorrect.

This PR removes the old behavior and uses a simple time
limit instead for throttling.  There is a maximum number
of MOUSEMOVE events per second (MOUSEMOVES_PER_SECOND)
and events that fail this limit are retyped to
INBETWEEN_MOUSEMOVE.
 2023-07-15 13:35:05 -07:00