Merge pull request #4462 from daschuer/effects_refactoring_1

Effects refactoring minor fixes
mixxxdj · Oct 21, 2021 · b96d24b · b96d24b
2 parents e817b69 + afc940c
commit b96d24b
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 0 deletions.
diff --git a/src/effects/backends/builtin/autopaneffect.h b/src/effects/backends/builtin/autopaneffect.h
@@ -20,6 +20,7 @@ class RampedSample {
     inline RampedSample()
             : ramped(false),
               maxDifference(1.0f),
+              currentValue(0),
               initialized(false) {
     }
 

diff --git a/src/effects/backends/builtin/reverbeffect.h b/src/effects/backends/builtin/reverbeffect.h
@@ -19,6 +19,7 @@ class ReverbGroupState : public EffectState {
   public:
     ReverbGroupState(const mixxx::EngineParameters& engineParameters)
             : EffectState(engineParameters),
+              sampleRate(engineParameters.sampleRate()),
               sendPrevious(0) {
     }
 

diff --git a/src/engine/effects/message.h b/src/engine/effects/message.h
@@ -168,4 +168,5 @@ class EffectsRequestHandler {
     virtual bool processEffectsRequest(
             EffectsRequest& message,
             EffectsResponsePipe* pResponsePipe) = 0;
+    virtual ~EffectsRequestHandler() = default;
 };