refactoring

git-svn-id: http://svn.code.sf.net/p/qmmp-dev/code/trunk/qmmp@10216 90c681e8-e032-0410-971d-27865f9a5e38

1 files changed, 18 insertions, 17 deletions

diff --git a/src/plugins/Ui/skinned/mainvisual.cpp b/src/plugins/Ui/skinned/mainvisual.cpp
index bd67cde32..45291a1e2 100644
--- a/src/plugins/Ui/skinned/mainvisual.cpp
+++ b/src/plugins/Ui/skinned/mainvisual.cpp
@@ -120,28 +120,29 @@ void MainVisual::showEvent (QShowEvent *)
 void MainVisual::mousePressEvent (QMouseEvent *e)
 {
     if (e->button() == Qt::RightButton)
+    {
         m_menu->exec(e->globalPos());
-    else
+        return;
+    }
+
+    m_pixmap = m_bg;
+    if (!m_vis)
+        setVisual(new mainvisual::Analyzer);
+    else if (m_vis->name() == "Analyzer")
+        setVisual(new mainvisual::Scope);
+    else if (m_vis->name() == "Scope")
+        setVisual(nullptr);
+
+    QString str = m_vis ? m_vis->name() : "Off";
+    for(QAction *act : m_visModeGroup->actions())
     {
-        m_pixmap = m_bg;
-        if (!m_vis)
-            setVisual(new mainvisual::Analyzer);
-        else if (m_vis->name() == "Analyzer")
-            setVisual(new mainvisual::Scope);
-        else if (m_vis->name() == "Scope")
-            setVisual(nullptr);
-
-        QString str = m_vis ? m_vis->name() : "Off";
-        for(QAction *act : m_visModeGroup->actions())
+        if (str == act->data().toString())
         {
-            if (str == act->data().toString())
-            {
-                act->setChecked(true);
-                break;
-            }
+            act->setChecked(true);
+            break;
         }
-        writeSettings();
     }
+    writeSettings();
 }
 
 void MainVisual::start()