Skip to content

Commit

Permalink
renamed tick conversion functions in NotationPlayback and PlaybackCon…
Browse files Browse the repository at this point in the history
…troller
  • Loading branch information
22justinl committed Jul 20, 2024
1 parent 69dda21 commit f3733e1
Show file tree
Hide file tree
Showing 5 changed files with 14 additions and 14 deletions.
2 changes: 1 addition & 1 deletion src/notation/inotationplayback.h
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ class INotationPlayback

virtual const Tempo& tempo(muse::midi::tick_t tick) const = 0;
virtual MeasureBeat beat(muse::midi::tick_t tick) const = 0;
virtual muse::midi::tick_t beatToTick(int measureIndex, int beatIndex) const = 0;
virtual muse::midi::tick_t beatToRawTick(int measureIndex, int beatIndex) const = 0;

virtual double tempoMultiplier() const = 0;
virtual void setTempoMultiplier(double multiplier) = 0;
Expand Down
2 changes: 1 addition & 1 deletion src/notation/internal/notationplayback.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -360,7 +360,7 @@ MeasureBeat NotationPlayback::beat(tick_t tick) const
return measureBeat;
}

tick_t NotationPlayback::beatToTick(int measureIndex, int beatIndex) const
tick_t NotationPlayback::beatToRawTick(int measureIndex, int beatIndex) const
{
return score() ? score()->sigmap()->bar2tick(measureIndex, beatIndex) : 0;
}
Expand Down
2 changes: 1 addition & 1 deletion src/notation/internal/notationplayback.h
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ class NotationPlayback : public INotationPlayback, public muse::async::Asyncable

const Tempo& tempo(muse::midi::tick_t tick) const override;
MeasureBeat beat(muse::midi::tick_t tick) const override;
muse::midi::tick_t beatToTick(int measureIndex, int beatIndex) const override;
muse::midi::tick_t beatToRawTick(int measureIndex, int beatIndex) const override;

double tempoMultiplier() const override;
void setTempoMultiplier(double multiplier) override;
Expand Down
18 changes: 9 additions & 9 deletions src/playback/internal/playbackcontroller.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -213,7 +213,7 @@ void PlaybackController::seek(const midi::tick_t tick)
return;
}

seek(tickToSecs(tick));
seek(playedTickToSecs(tick));
}

void PlaybackController::seek(const audio::secs_t secs)
Expand Down Expand Up @@ -635,7 +635,7 @@ msecs_t PlaybackController::playbackStartMsecs() const
if (!startTick.ret) {
return 0;
}
return tickToMsecs(startTick.val);
return playedTickToMsecs(startTick.val);
}

return 0;
Expand Down Expand Up @@ -797,8 +797,8 @@ void PlaybackController::updateLoop()
return;
}

msecs_t fromMsecs = tickToMsecs(playbackTickFrom.val);
msecs_t toMsecs = tickToMsecs(playbackTickTo.val);
msecs_t fromMsecs = playedTickToMsecs(playbackTickFrom.val);
msecs_t toMsecs = playedTickToMsecs(playbackTickTo.val);
currentPlayer()->setLoop(fromMsecs, toMsecs);

enableLoop();
Expand Down Expand Up @@ -1434,7 +1434,7 @@ MeasureBeat PlaybackController::currentBeat() const

msecs_t PlaybackController::beatToMilliseconds(int measureIndex, int beatIndex) const
{
return notationPlayback() ? tickToMsecs(notationPlayback()->beatToTick(measureIndex, beatIndex)) : 0;
return notationPlayback() ? playedTickToMsecs(notationPlayback()->beatToRawTick(measureIndex, beatIndex)) : 0;
}

secs_t PlaybackController::beatToSecs(int measureIndex, int beatIndex) const
Expand All @@ -1443,10 +1443,10 @@ secs_t PlaybackController::beatToSecs(int measureIndex, int beatIndex) const
return 0;
}

muse::midi::tick_t rawTick = notationPlayback()->beatToTick(measureIndex, beatIndex);
muse::midi::tick_t rawTick = notationPlayback()->beatToRawTick(measureIndex, beatIndex);
muse::midi::tick_t playedTick = notationPlayback()->playPositionTickByRawTick(rawTick).val;

return tickToSecs(playedTick);
return playedTickToSecs(playedTick);
}

double PlaybackController::tempoMultiplier() const
Expand Down Expand Up @@ -1588,13 +1588,13 @@ bool PlaybackController::canReceiveAction(const ActionCode&) const
return m_masterNotation != nullptr && m_masterNotation->hasParts();
}

msecs_t PlaybackController::tickToMsecs(int tick) const
msecs_t PlaybackController::playedTickToMsecs(int tick) const
{
float sec = notationPlayback()->playedTickToSec(tick);
return secondsToMilliseconds(sec);
}

muse::audio::secs_t PlaybackController::tickToSecs(int tick) const
muse::audio::secs_t PlaybackController::playedTickToSecs(int tick) const
{
return secs_t(notationPlayback()->playedTickToSec(tick));
}
4 changes: 2 additions & 2 deletions src/playback/internal/playbackcontroller.h
Original file line number Diff line number Diff line change
Expand Up @@ -208,8 +208,8 @@ class PlaybackController : public IPlaybackController, public muse::actions::Act
void removeNonExistingTracks();
void removeTrack(const engraving::InstrumentTrackId& instrumentTrackId);

muse::audio::msecs_t tickToMsecs(int tick) const;
muse::audio::secs_t tickToSecs(int tick) const;
muse::audio::msecs_t playedTickToMsecs(int tick) const;
muse::audio::secs_t playedTickToSecs(int tick) const;

notation::INotationPtr m_notation;
notation::IMasterNotationPtr m_masterNotation;
Expand Down

0 comments on commit f3733e1

Please sign in to comment.