From fd6432c7271b0715e710a8c9c8bd3066b56f7222 Mon Sep 17 00:00:00 2001 From: Romero Malaquias Date: Tue, 8 Dec 2015 23:10:16 -0300 Subject: [PATCH] Avoiding conditional directives that split up parts os statements --- src/notify.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/notify.c b/src/notify.c index f6a2fcf..8cfade9 100644 --- a/src/notify.c +++ b/src/notify.c @@ -234,17 +234,17 @@ void* do_playing(void* _p) } bool has_looping = false; + bool test_active_notify = false; int i; for (i = 0; i < ACTIVE_NOTIFS_MAX; i ++) { if (actives[i].looping) has_looping = true; - - if (actives[i].active && !actives[i].looping - #ifdef BOX_NOTIFY - && !actives[i].box - #endif - ) { + test_active_notify = actives[i].active && !actives[i].looping; + #ifdef BOX_NOTIFY + test_active_notify = test_active_notify && !actives[i].box; + #endif + if (test_active_notify) { if(actives[i].id_indicator) *actives[i].id_indicator = -1; /* reset indicator value */