bug fix: clip state not correctly reset when changing servers

ChangeLog

@@ -20,18 +20,15 @@
 - bug fix: grouping faders in the client should be proportional (see discussion in #202, #419)
 
 
-
 TODO bug fix: incorrect selection of UI language (#408) !!!!!!!!!!!!!!!!!!!!!!!!!!!!!
      -> note that for the 3.5.8 bug fix release we went back to the original translation code (e.g. no pt_BR!)
 
-TODO improve settings management -> move settings class in client/server classes, move actual settings variables
-
 TODO add new register message which contains version and, e.g., max number of clients
 
 TODO Inconsistency between Input meter and Audio mixer meter #423
 
-TODO https://github.com/corrados/jamulus/issues/341#issuecomment-647172946
-     - generate .qm on compile time with lrelease
+TODO improve settings management -> move settings class in client/server classes, move actual settings variables
+
 
 
 

src/levelmeter.cpp

@@ -246,8 +246,14 @@ void CLevelMeter::SetValue ( const double dValue )
 
         TimerClip.start();
     }
-    else if ( !TimerClip.isActive() )
-    {
+}
+
+void CLevelMeter::ClipReset()
+{
+    // we manually want to reset the clipping indicator: stop timer and reset
+    // clipping indicator GUI element
+    TimerClip.stop();
+
     switch ( eLevelMeterType )
     {
     case MT_LED:
@@ -259,15 +265,6 @@ void CLevelMeter::SetValue ( const double dValue )
         SetBarMeterStyleAndClipStatus ( eLevelMeterType, false );
         break;
     }
-    }
-}
-
-void CLevelMeter::ClipReset()
-{
-    if ( eLevelMeterType == MT_LED )
-    {
-        vecpLEDs[NUM_STEPS_LED_BAR]->SetColor ( cLED::RL_BLACK );
-    }
 }