Don't do the preloaded check on manual track changes.

Update issue #332
Status: fixed
Thanks for the patch!  Calling Stop() in there meant that the OSD got shown
briefly with a "Stopped" message though, so I've changed GstEngine::Load
instead - the problem was with the code that checked whether the pipeline had
already started to load that URL as a result of GstEngine::StartPreloading.
It'll now only do that on an automatic track change.
This commit is contained in:
David Sansome 2010-05-22 17:19:27 +00:00
parent aff901e44d
commit d054e06444
1 changed files with 2 additions and 1 deletions

View File

@ -384,7 +384,8 @@ bool GstEngine::Load(const QUrl& url, Engine::TrackChangeType change) {
((crossfade_enabled_ && change == Engine::Manual) ||
(autocrossfade_enabled_ && change == Engine::Auto));
if (!crossfade && current_pipeline_ && current_pipeline_->url() == url) {
if (!crossfade && current_pipeline_ && current_pipeline_->url() == url &&
change == Engine::Auto) {
// We're not crossfading, and the pipeline is already playing the URI we
// want, so just do nothing.
return true;