From 4b43bcadfbdc7be4fe4f98ed8605ad04284246e1 Mon Sep 17 00:00:00 2001
From: Jan Holthuis <jholthuis@mixxx.org>
Date: Tue, 6 Jul 2021 17:34:46 +0200
Subject: [PATCH 1/2] Track: Remove CuePosition class and corresponding methods

---
 src/track/cue.cpp |  4 ----
 src/track/cue.h   | 39 ---------------------------------------
 src/track/track.h |  8 --------
 3 files changed, 51 deletions(-)

diff --git a/src/track/cue.cpp b/src/track/cue.cpp
index bb63a77a1c2..6816c921f5e 100644
--- a/src/track/cue.cpp
+++ b/src/track/cue.cpp
@@ -273,7 +273,3 @@ mixxx::audio::FramePos Cue::getEndPosition() const {
     QMutexLocker lock(&m_mutex);
     return m_endPosition;
 }
-
-bool operator==(const CuePosition& lhs, const CuePosition& rhs) {
-    return lhs.getPosition() == rhs.getPosition();
-}
diff --git a/src/track/cue.h b/src/track/cue.h
index 1b469661af4..f84000ada15 100644
--- a/src/track/cue.h
+++ b/src/track/cue.h
@@ -145,42 +145,3 @@ class CuePointer : public std::shared_ptr<Cue> {
   private:
     static void deleteLater(Cue* pCue);
 };
-
-class CuePosition {
-  public:
-    CuePosition()
-            : m_position(0.0) {
-    }
-    CuePosition(double position)
-            : m_position(position) {
-    }
-
-    double getPosition() const {
-        return m_position;
-    }
-
-    void setPosition(double position) {
-        m_position = position;
-    }
-
-    void set(double position) {
-        m_position = position;
-    }
-
-    void reset() {
-        m_position = 0.0;
-    }
-
-  private:
-    double m_position;
-};
-
-bool operator==(const CuePosition& lhs, const CuePosition& rhs);
-
-inline bool operator!=(const CuePosition& lhs, const CuePosition& rhs) {
-    return !(lhs == rhs);
-}
-
-inline QDebug operator<<(QDebug dbg, const CuePosition& arg) {
-    return dbg << "position =" << arg.getPosition();
-}
diff --git a/src/track/track.h b/src/track/track.h
index 9d6a95dbbfb..5dab8dd708f 100644
--- a/src/track/track.h
+++ b/src/track/track.h
@@ -263,16 +263,8 @@ class Track : public QObject {
 
     /// Get the track's main cue point
     mixxx::audio::FramePos getMainCuePosition() const;
-    CuePosition getCuePoint() const {
-        return getMainCuePosition().toEngineSamplePosMaybeInvalid();
-    };
     // Set the track's main cue point
     void setMainCuePosition(mixxx::audio::FramePos position);
-    void setCuePoint(CuePosition position) {
-        setMainCuePosition(
-                mixxx::audio::FramePos::fromEngineSamplePosMaybeInvalid(
-                        position.getPosition()));
-    }
     /// Shift all cues by a constant offset
     void shiftCuePositionsMillis(mixxx::audio::FrameDiff_t milliseconds);
     // Call when analysis is done.

From 004fc486d0453e4d93765af720b77b9c3bf3188f Mon Sep 17 00:00:00 2001
From: Jan Holthuis <jholthuis@mixxx.org>
Date: Tue, 6 Jul 2021 17:39:07 +0200
Subject: [PATCH 2/2] Cue: Remove all sample-based methods

These methods are now unused.
---
 src/track/cue.h | 17 -----------------
 1 file changed, 17 deletions(-)

diff --git a/src/track/cue.h b/src/track/cue.h
index f84000ada15..69536439bce 100644
--- a/src/track/cue.h
+++ b/src/track/cue.h
@@ -68,27 +68,10 @@ class Cue : public QObject {
 
     mixxx::audio::FramePos getPosition() const;
     void setStartPosition(mixxx::audio::FramePos position);
-    void setStartPosition(double samplePosition) {
-        setStartPosition(mixxx::audio::FramePos::fromEngineSamplePosMaybeInvalid(samplePosition));
-    }
     void setEndPosition(mixxx::audio::FramePos position);
-    void setEndPosition(double samplePosition) {
-        setEndPosition(mixxx::audio::FramePos::fromEngineSamplePosMaybeInvalid(samplePosition));
-    }
     void setStartAndEndPosition(
             mixxx::audio::FramePos startPosition,
             mixxx::audio::FramePos endPosition);
-    void setStartAndEndPosition(
-            double sampleStartPosition,
-            double 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);
 
     mixxx::audio::FrameDiff_t getLengthFrames() const;