griefith/test2
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Cleanup: simplify WM_event_timer_sleep

Browse Source 
Use remove the inline loop as it was only used to check if the timer
is in wm->timers.
This commit is contained in:
Campbell Barton
2023-08-30 11:09:26 +10:00
parent 745f78f9f0
commit 09471c8636
1 changed files with 9 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
source/blender/windowmanager/intern/wm_window.cc
View File

@@ -1950,15 +1950,16 @@ eWM_CapabilitiesFlag WM_capabilities_flag()
void WM_event_timer_sleep(wmWindowManager *wm, wmWindow * /*win*/, wmTimer *timer, bool do_sleep)
{
LISTBASE_FOREACH (wmTimer *, wt, &wm->timers) {
if (wt->flags & WM_TIMER_TAGGED_FOR_REMOVAL) {
continue;
}
if (wt == timer) {
wt->sleep = do_sleep;
break;
}
/* Extra security check. */
if (BLI_findindex(&wm->timers, timer) == -1) {
return;
}
/* It's disputable if this is needed, when tagged for removal,
* the sleep value won't be used anyway. */
if (timer->flags & WM_TIMER_TAGGED_FOR_REMOVAL) {
return;
}
timer->sleep = do_sleep;
}
wmTimer *WM_event_timer_add(wmWindowManager *wm, wmWindow *win, int event_type, double timestep)