Cue: Use FramePos::fromEngineSamplePosMaybeInvalid in legacy methods

mixxxdj · Jul 7, 2021 · 7a89aed · 7a89aed
1 parent c86cbb6
commit 7a89aed
Showing 1 changed file with 8 additions and 18 deletions.
diff --git a/src/track/cue.h b/src/track/cue.h
@@ -69,34 +69,24 @@ class Cue : public QObject {
     mixxx::audio::FramePos getPosition() const;
     void setStartPosition(mixxx::audio::FramePos position);
     void setStartPosition(double samplePosition) {
-        mixxx::audio::FramePos position;
-        if (samplePosition != Cue::kNoPosition) {
-            position = mixxx::audio::FramePos::fromEngineSamplePos(samplePosition);
-        }
-        setStartPosition(position);
+        setStartPosition(mixxx::audio::FramePos::fromEngineSamplePosMaybeInvalid(samplePosition));
     }
     void setEndPosition(mixxx::audio::FramePos position);
     void setEndPosition(double samplePosition) {
-        mixxx::audio::FramePos position;
-        if (samplePosition != Cue::kNoPosition) {
-            position = mixxx::audio::FramePos::fromEngineSamplePos(samplePosition);
-        }
-        setEndPosition(position);
+        setEndPosition(mixxx::audio::FramePos::fromEngineSamplePosMaybeInvalid(samplePosition));
     }
     void setStartAndEndPosition(
             mixxx::audio::FramePos startPosition,
             mixxx::audio::FramePos endPosition);
     void setStartAndEndPosition(
             double sampleStartPosition,
             double sampleEndPosition) {
-        mixxx::audio::FramePos startPosition;
-        mixxx::audio::FramePos endPosition;
-        if (sampleStartPosition != Cue::kNoPosition) {
-            startPosition = mixxx::audio::FramePos::fromEngineSamplePos(sampleStartPosition);
-        }
-        if (sampleEndPosition != Cue::kNoPosition) {
-            endPosition = mixxx::audio::FramePos::fromEngineSamplePos(sampleEndPosition);
-        }
+        const auto startPosition =
+                mixxx::audio::FramePos::fromEngineSamplePosMaybeInvalid(
+                        sampleStartPosition);
+        const auto endPosition =
+                mixxx::audio::FramePos::fromEngineSamplePosMaybeInvalid(
+                        sampleEndPosition);
         setStartAndEndPosition(startPosition, endPosition);
     }
     void shiftPositionFrames(mixxx::audio::FrameDiff_t frameOffset);