From dd00d3bc66514f6d6ffa61906efda00f203e6911 Mon Sep 17 00:00:00 2001
From: Eknous-P <ehovakimyan053@gmail.com>
Date: Fri, 28 Jul 2023 18:39:40 +0400
Subject: [PATCH] cleanup

---
 src/gui/effectList.cpp | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/src/gui/effectList.cpp b/src/gui/effectList.cpp
index e9da54d56..f6b4c6c31 100644
--- a/src/gui/effectList.cpp
+++ b/src/gui/effectList.cpp
@@ -23,6 +23,7 @@ void FurnaceGUI::drawEffectList() {
       ImGui::Checkbox("System (primary)",&effectShowSysPrimary);
       ImGui::Checkbox("System (secondary)",&effectShowSysSecondary);
       ImGui::Checkbox("Miscellaneous",&effectShowMisc);
+
       if (ImGui::Button("All")) {
         effectShowMisc = true;
         effectShowPanning = true;
@@ -46,6 +47,7 @@ void FurnaceGUI::drawEffectList() {
         effectShowSysPrimary = false;
         effectShowSysSecondary = false;
       }
+
       ImGui::EndPopup();
     }
     
@@ -63,6 +65,10 @@ void FurnaceGUI::drawEffectList() {
       for (int i=0; i<256; i++) {
         const char* name=e->getEffectDesc(i,cursor.xCoarse);
         bool effectShow = true;
+        if (name==prevName) {
+          continue;
+        }
+        prevName=name;
         switch (fxColors[i]) {
           case GUI_COLOR_PATTERN_EFFECT_MISC: effectShow = effectShowMisc; break;
           case GUI_COLOR_PATTERN_EFFECT_SONG: effectShow = effectShowSong; break;
@@ -75,10 +81,6 @@ void FurnaceGUI::drawEffectList() {
           case GUI_COLOR_PATTERN_EFFECT_SYS_SECONDARY: effectShow = effectShowSysSecondary; break;
           default: effectShow = true; break;
         }
-        if (name==prevName) {
-          continue;
-        }
-        prevName=name;
         if (name!=NULL && effectShow) {
           ImGui::TableNextRow();
           ImGui::TableNextColumn();