GstEngine: Simplify use of `State`

This commit is contained in:
Jonas Kvinge 2024-08-09 19:26:15 +02:00
parent d5d2eaba8a
commit fe30f27af3
2 changed files with 15 additions and 15 deletions

View File

@ -149,19 +149,19 @@ bool GstEngine::Init() {
EngineBase::State GstEngine::state() const { EngineBase::State GstEngine::state() const {
if (!current_pipeline_) return stream_url_.isEmpty() ? EngineBase::State::Empty : EngineBase::State::Idle; if (!current_pipeline_) return stream_url_.isEmpty() ? State::Empty : State::Idle;
switch (current_pipeline_->state()) { switch (current_pipeline_->state()) {
case GST_STATE_NULL: case GST_STATE_NULL:
return EngineBase::State::Empty; return State::Empty;
case GST_STATE_READY: case GST_STATE_READY:
return EngineBase::State::Idle; return State::Idle;
case GST_STATE_PLAYING: case GST_STATE_PLAYING:
return EngineBase::State::Playing; return State::Playing;
case GST_STATE_PAUSED: case GST_STATE_PAUSED:
return EngineBase::State::Paused; return State::Paused;
default: default:
return EngineBase::State::Empty; return State::Empty;
} }
} }
@ -289,7 +289,7 @@ void GstEngine::Stop(const bool stop_after) {
current_pipeline_.reset(); current_pipeline_.reset();
BufferingFinished(); BufferingFinished();
emit StateChanged(EngineBase::State::Empty); emit StateChanged(State::Empty);
} }
@ -303,7 +303,7 @@ void GstEngine::Pause() {
current_pipeline_->StartFader(fadeout_pause_duration_nanosec_, QTimeLine::Forward, QEasingCurve::InOutQuad, false); current_pipeline_->StartFader(fadeout_pause_duration_nanosec_, QTimeLine::Forward, QEasingCurve::InOutQuad, false);
is_fading_out_to_pause_ = false; is_fading_out_to_pause_ = false;
has_faded_out_ = false; has_faded_out_ = false;
emit StateChanged(EngineBase::State::Playing); emit StateChanged(State::Playing);
return; return;
} }
@ -313,7 +313,7 @@ void GstEngine::Pause() {
} }
else { else {
current_pipeline_->SetState(GST_STATE_PAUSED); current_pipeline_->SetState(GST_STATE_PAUSED);
emit StateChanged(EngineBase::State::Paused); emit StateChanged(State::Paused);
StopTimers(); StopTimers();
} }
} }
@ -335,7 +335,7 @@ void GstEngine::Unpause() {
has_faded_out_ = false; has_faded_out_ = false;
} }
emit StateChanged(EngineBase::State::Playing); emit StateChanged(State::Playing);
StartTimers(); StartTimers();
} }
@ -576,7 +576,7 @@ void GstEngine::HandlePipelineError(const int pipeline_id, const int domain, con
current_pipeline_.reset(); current_pipeline_.reset();
BufferingFinished(); BufferingFinished();
emit StateChanged(EngineBase::State::Error); emit StateChanged(State::Error);
if ( if (
(domain == static_cast<int>(GST_RESOURCE_ERROR) && ( (domain == static_cast<int>(GST_RESOURCE_ERROR) && (
@ -632,7 +632,7 @@ void GstEngine::FadeoutPauseFinished() {
fadeout_pause_pipeline_->SetState(GST_STATE_PAUSED); fadeout_pause_pipeline_->SetState(GST_STATE_PAUSED);
current_pipeline_->SetState(GST_STATE_PAUSED); current_pipeline_->SetState(GST_STATE_PAUSED);
emit StateChanged(EngineBase::State::Paused); emit StateChanged(State::Paused);
StopTimers(); StopTimers();
is_fading_out_to_pause_ = false; is_fading_out_to_pause_ = false;
@ -687,7 +687,7 @@ void GstEngine::PlayDone(const GstStateChangeReturn ret, const quint64 offset_na
Seek(offset_nanosec); Seek(offset_nanosec);
} }
emit StateChanged(EngineBase::State::Playing); emit StateChanged(State::Playing);
// We've successfully started playing a media stream with this url // We've successfully started playing a media stream with this url
emit ValidSongRequested(stream_url_); emit ValidSongRequested(stream_url_);
@ -848,7 +848,7 @@ SharedPtr<GstEnginePipeline> GstEngine::CreatePipeline(const QUrl &media_url, co
if (!ret->InitFromUrl(media_url, stream_url, gst_url, end_nanosec, ebur128_loudness_normalizing_gain_db, error)) { if (!ret->InitFromUrl(media_url, stream_url, gst_url, end_nanosec, ebur128_loudness_normalizing_gain_db, error)) {
ret.reset(); ret.reset();
emit Error(error); emit Error(error);
emit StateChanged(EngineBase::State::Error); emit StateChanged(State::Error);
emit FatalError(); emit FatalError();
} }

View File

@ -60,7 +60,7 @@ class GstEngine : public EngineBase, public GstBufferConsumer {
Type type() const override { return Type::GStreamer; } Type type() const override { return Type::GStreamer; }
bool Init() override; bool Init() override;
EngineBase::State state() const override; State state() const override;
void StartPreloading(const QUrl &media_url, const QUrl &stream_url, const bool force_stop_at_end, const qint64 beginning_nanosec, const qint64 end_nanosec) override; void StartPreloading(const QUrl &media_url, const QUrl &stream_url, const bool force_stop_at_end, const qint64 beginning_nanosec, const qint64 end_nanosec) override;
bool Load(const QUrl &media_url, const QUrl &stream_url, const EngineBase::TrackChangeFlags change, const bool force_stop_at_end, const quint64 beginning_nanosec, const qint64 end_nanosec, const std::optional<double> ebur128_integrated_loudness_lufs) override; bool Load(const QUrl &media_url, const QUrl &stream_url, const EngineBase::TrackChangeFlags change, const bool force_stop_at_end, const quint64 beginning_nanosec, const qint64 end_nanosec, const std::optional<double> ebur128_integrated_loudness_lufs) override;
bool Play(const quint64 offset_nanosec) override; bool Play(const quint64 offset_nanosec) override;