BeatGrid: Remove all usage ouside of the Beats class

mixxxdj · Sep 9, 2021 · 39222ac · 39222ac
1 parent efa3e80
commit 39222ac
Show file tree

Hide file tree

Showing 11 changed files with 222 additions and 228 deletions.
diff --git a/src/analyzer/analyzerbeats.cpp b/src/analyzer/analyzerbeats.cpp
@@ -10,7 +10,6 @@
 #include "analyzer/plugins/analyzersoundtouchbeats.h"
 #include "library/rekordbox/rekordboxconstants.h"
 #include "track/beatfactory.h"
-#include "track/beatgrid.h"
 #include "track/beatutils.h"
 #include "track/track.h"
 
@@ -234,7 +233,7 @@ void AnalyzerBeats::storeResults(TrackPointer pTrack) {
     } else {
         mixxx::Bpm bpm = m_pPlugin->getBpm();
         qDebug() << "AnalyzerBeats plugin detected constant BPM: " << bpm;
-        pBeats = mixxx::BeatGrid::makeBeatGrid(m_sampleRate, bpm, mixxx::audio::kStartFramePos);
+        pBeats = mixxx::Beats::fromConstTempo(m_sampleRate, mixxx::audio::kStartFramePos, bpm);
     }
 
     pTrack->trySetBeats(pBeats);

diff --git a/src/library/dao/trackdao.cpp b/src/library/dao/trackdao.cpp
@@ -24,7 +24,6 @@
 #include "library/trackset/crate/cratestorage.h"
 #include "moc_trackdao.cpp"
 #include "sources/soundsourceproxy.h"
-#include "track/beatgrid.h"
 #include "track/beats.h"
 #include "track/globaltrackcache.h"
 #include "track/keyfactory.h"
@@ -1245,8 +1244,8 @@ bool setTrackBeats(const QSqlRecord& record, const int column, Track* pTrack) {
         }
     } else if (bpm.isValid()) {
         // Load a temporary beat grid without offset that will be replaced by the analyzer.
-        const auto pBeats = mixxx::BeatGrid::makeBeatGrid(
-                pTrack->getSampleRate(), bpm, mixxx::audio::kStartFramePos);
+        const auto pBeats = mixxx::Beats::fromConstTempo(
+                pTrack->getSampleRate(), mixxx::audio::kStartFramePos, bpm);
         pTrack->trySetBeats(pBeats);
     } else {
         pTrack->trySetBeats(nullptr);

diff --git a/src/library/dlgtrackinfo.cpp b/src/library/dlgtrackinfo.cpp
@@ -14,7 +14,6 @@
 #include "moc_dlgtrackinfo.cpp"
 #include "preferences/colorpalettesettings.h"
 #include "sources/soundsourceproxy.h"
-#include "track/beatgrid.h"
 #include "track/beatutils.h"
 #include "track/keyfactory.h"
 #include "track/keyutils.h"
@@ -573,10 +572,10 @@ void DlgTrackInfo::slotBpmConstChanged(int state) {
             // The cue point should be set on a beat, so this seems
             // to be a good alternative
             const mixxx::audio::FramePos cuePosition = m_pLoadedTrack->getMainCuePosition();
-            m_pBeatsClone = mixxx::BeatGrid::makeBeatGrid(
+            m_pBeatsClone = mixxx::Beats::fromConstTempo(
                     m_pLoadedTrack->getSampleRate(),
-                    bpm,
-                    cuePosition);
+                    cuePosition,
+                    bpm);
         } else {
             m_pBeatsClone.reset();
         }
@@ -612,10 +611,10 @@ void DlgTrackInfo::slotSpinBpmValueChanged(double value) {
 
     if (!m_pBeatsClone) {
         const mixxx::audio::FramePos cuePosition = m_pLoadedTrack->getMainCuePosition();
-        m_pBeatsClone = mixxx::BeatGrid::makeBeatGrid(
+        m_pBeatsClone = mixxx::Beats::fromConstTempo(
                 m_pLoadedTrack->getSampleRate(),
-                bpm,
-                cuePosition);
+                cuePosition,
+                bpm);
     }
 
     const mixxx::Bpm oldValue = m_pBeatsClone->getBpm();

diff --git a/src/test/beatgridtest.cpp b/src/test/beatgridtest.cpp
@@ -2,7 +2,7 @@
 
 #include <QtDebug>
 
-#include "track/beatgrid.h"
+#include "track/beats.h"
 #include "track/track.h"
 #include "util/memory.h"
 
@@ -29,9 +29,9 @@ TEST(BeatGridTest, Scale) {
     constexpr mixxx::Bpm bpm(60.0);
     pTrack->trySetBpm(bpm.value());
 
-    auto pGrid = BeatGrid::makeBeatGrid(pTrack->getSampleRate(),
-            mixxx::Bpm(bpm),
-            mixxx::audio::kStartFramePos);
+    auto pGrid = Beats::fromConstTempo(pTrack->getSampleRate(),
+            mixxx::audio::kStartFramePos,
+            mixxx::Bpm(bpm));
 
     EXPECT_DOUBLE_EQ(bpm.value(), pGrid->getBpm().value());
     pGrid = pGrid->scale(Beats::BpmScale::Double);
@@ -61,9 +61,9 @@ TEST(BeatGridTest, TestNthBeatWhenOnBeat) {
     pTrack->trySetBpm(bpm);
     constexpr mixxx::audio::FrameDiff_t beatLengthFrames = 60.0 * sampleRate / bpm;
 
-    auto pGrid = BeatGrid::makeBeatGrid(pTrack->getSampleRate(),
-            mixxx::Bpm(bpm),
-            mixxx::audio::kStartFramePos);
+    auto pGrid = Beats::fromConstTempo(pTrack->getSampleRate(),
+            mixxx::audio::kStartFramePos,
+            mixxx::Bpm(bpm));
     // Pretend we're on the 20th beat;
     constexpr mixxx::audio::FramePos position(beatLengthFrames * 20);
 
@@ -105,9 +105,9 @@ TEST(BeatGridTest, TestNthBeatWhenNotOnBeat) {
     pTrack->trySetBpm(bpm.value());
     const mixxx::audio::FrameDiff_t beatLengthFrames = 60.0 * sampleRate / bpm.value();
 
-    auto pGrid = BeatGrid::makeBeatGrid(pTrack->getSampleRate(),
-            bpm,
-            mixxx::audio::kStartFramePos);
+    auto pGrid = Beats::fromConstTempo(pTrack->getSampleRate(),
+            mixxx::audio::kStartFramePos,
+            bpm);
 
     // Pretend we're half way between the 20th and 21st beat
     const mixxx::audio::FramePos previousBeat(beatLengthFrames * 20.0);

diff --git a/src/test/beatstranslatetest.cpp b/src/test/beatstranslatetest.cpp
@@ -1,5 +1,5 @@
 #include "test/mockedenginebackendtest.h"
-#include "track/beatgrid.h"
+#include "track/beats.h"
 #include "util/memory.h"
 
 class BeatsTranslateTest : public MockedEngineBackendTest {
@@ -9,14 +9,14 @@ TEST_F(BeatsTranslateTest, SimpleTranslateMatch) {
     // Set up BeatGrids for decks 1 and 2.
     const auto bpm = mixxx::Bpm(60.0);
     constexpr auto firstBeat = mixxx::audio::kStartFramePos;
-    auto grid1 = mixxx::BeatGrid::makeBeatGrid(
-            m_pTrack1->getSampleRate(), bpm, firstBeat);
+    auto grid1 = mixxx::Beats::fromConstTempo(
+            m_pTrack1->getSampleRate(), firstBeat, bpm);
     m_pTrack1->trySetBeats(grid1);
     ASSERT_DOUBLE_EQ(firstBeat.value(),
             grid1->findClosestBeat(mixxx::audio::kStartFramePos).value());
 
-    auto grid2 = mixxx::BeatGrid::makeBeatGrid(
-            m_pTrack2->getSampleRate(), bpm, firstBeat);
+    auto grid2 = mixxx::Beats::fromConstTempo(
+            m_pTrack2->getSampleRate(), firstBeat, bpm);
     m_pTrack2->trySetBeats(grid2);
     ASSERT_DOUBLE_EQ(firstBeat.value(),
             grid2->findClosestBeat(mixxx::audio::kStartFramePos).value());

diff --git a/src/test/bpmcontrol_test.cpp b/src/test/bpmcontrol_test.cpp
@@ -1,14 +1,14 @@
+#include "engine/controls/bpmcontrol.h"
+
 #include <gtest/gtest.h>
 
-#include <QtDebug>
 #include <QScopedPointer>
+#include <QtDebug>
 
-#include "mixxxtest.h"
 #include "control/controlobject.h"
 #include "control/controlpushbutton.h"
-#include "engine/controls/bpmcontrol.h"
-#include "track/beatgrid.h"
-#include "track/beatmap.h"
+#include "mixxxtest.h"
+#include "track/beats.h"
 #include "track/track.h"
 
 class BpmControlTest : public MixxxTest {
@@ -35,8 +35,8 @@ TEST_F(BpmControlTest, BeatContext_BeatGrid) {
     const auto bpm = mixxx::Bpm(60.0);
     const mixxx::audio::FrameDiff_t expectedBeatLengthFrames = (60.0 * sampleRate / bpm.value());
 
-    const mixxx::BeatsPointer pBeats = mixxx::BeatGrid::makeBeatGrid(
-            pTrack->getSampleRate(), bpm, mixxx::audio::kStartFramePos);
+    const mixxx::BeatsPointer pBeats = mixxx::Beats::fromConstTempo(
+            pTrack->getSampleRate(), mixxx::audio::kStartFramePos, bpm);
 
     // On a beat.
     mixxx::audio::FramePos prevBeatPosition;