Avoid unneeded locking

2 files changed, 4 insertions, 6 deletions

diff --git a/src/track/track.cpp b/src/track/track.cpp
index 8a4c321681..41d8984282 100644
--- a/src/track/track.cpp
+++ b/src/track/track.cpp
@@ -877,11 +877,6 @@ void Track::removeCuesOfType(mixxx::CueType type) {
     }
 }
 
-QList<CuePointer> Track::getCuePoints() const {
-    QMutexLocker lock(&m_qMutex);
-    return m_cuePoints;
-}
-
 void Track::setCuePoints(const QList<CuePointer>& cuePoints) {
     // While this method could be called from any thread,
     // associated Cue objects should always live on the
diff --git a/src/track/track.h b/src/track/track.h
index 8d4f8e33a5..24a3466217 100644
--- a/src/track/track.h
+++ b/src/track/track.h
@@ -268,7 +268,10 @@ class Track : public QObject {
     CuePointer findCueById(int id) const;
     void removeCue(const CuePointer& pCue);
     void removeCuesOfType(mixxx::CueType);
-    QList<CuePointer> getCuePoints() const;
+    QList<CuePointer> getCuePoints() const {
+        // Copying implicitly shared collections is thread-safe
+        return m_cuePoints;
+    }
 
     void setCuePoints(const QList<CuePointer>& cuePoints);