From cd7edbcc11a8b180dccdd584106c87736f9bcb3b Mon Sep 17 00:00:00 2001 From: Iain Patterson Date: Fri, 22 Jun 2012 17:03:59 +0100 Subject: [PATCH] Correct window shading logic. Fons Adriaensen reported that: > WM's 'unshade' seems to use multiple Expose events [...] even when > all animations and 'superfluous effects' are disabled. Typo in shade_animate() could cause windows to be animated when the no_animations preference was set. With this patch I see fewer XMoveWindow() calls and quicker shading. --- src/actions.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/actions.c b/src/actions.c index 61be5353..fed1c1e8 100644 --- a/src/actions.c +++ b/src/actions.c @@ -1892,7 +1892,7 @@ static void shade_animate(WWindow *wwin, Bool what) int y, s, w, h; time_t time0 = time(NULL); - if (wwin->flags.skip_next_animation && wPreferences.no_animations) + if (wwin->flags.skip_next_animation || wPreferences.no_animations) return; switch(what) {