Revert "Merge pull request mixxxdj#11052 from ronso0/vumetergl-init-p…

…aint-fix"

This reverts commit dfa313f, reversing
changes made to dba50e2.

src/widget/wvumetergl.cpp

@@ -16,7 +16,7 @@
 WVuMeterGL::WVuMeterGL(QWidget* parent)
         : QGLWidget(parent, SharedGLContext::getWidget()),
           WBaseWidget(this),
-          m_iPendingRenders(0),
+          m_bHasRendered(false),
           m_bSwapNeeded(false),
           m_dParameter(0),
           m_dPeakParameter(0),
@@ -156,10 +156,10 @@ void WVuMeterGL::updateState(mixxx::Duration elapsed) {
 
 void WVuMeterGL::paintEvent(QPaintEvent* e) {
     Q_UNUSED(e);
-    // Force rerendering when render is called from the vsync thread, e.g. to
+    // Force a rerender when render is called from the vsync thread, e.g. to
     // git rid artifacts after hiding and showing the mixer or incomplete
-    // initial drawing. Use 2 passes, in case triple buffering is used.
-    m_iPendingRenders = 2;
+    // initial drawing.
+    m_bHasRendered = false;
 }
 
 void WVuMeterGL::showEvent(QShowEvent* e) {
@@ -173,11 +173,8 @@ void WVuMeterGL::render(VSyncThread* vSyncThread) {
 
     updateState(vSyncThread->sinceLastSwap());
 
-    if (m_dParameter != m_dLastParameter || m_dPeakParameter != m_dLastPeakParameter) {
-        m_iPendingRenders = 2;
-    }
-
-    if (m_iPendingRenders == 0) {
+    if (m_bHasRendered && m_dParameter == m_dLastParameter &&
+            m_dPeakParameter == m_dLastPeakParameter) {
         return;
     }
 
@@ -297,7 +294,7 @@ void WVuMeterGL::render(VSyncThread* vSyncThread) {
 
     m_dLastParameter = m_dParameter;
     m_dLastPeakParameter = m_dPeakParameter;
-    m_iPendingRenders--;
+    m_bHasRendered = true;
     m_bSwapNeeded = true;
 }
 

src/widget/wvumetergl.h

@@ -38,9 +38,8 @@ class WVuMeterGL : public QGLWidget, public WBaseWidget {
     void showEvent(QShowEvent* /*unused*/) override;
     void setPeak(double parameter);
 
-    // To make sure we render at least N times even when we have no signal,
-    // for example after showEvent()
-    int m_iPendingRenders;
+    // To make sure we render at least once even when we have no signal
+    bool m_bHasRendered;
     // To indicate that we rendered so we need to swap
     bool m_bSwapNeeded;
     // Current parameter and peak parameter.