[prev in list] [next in list] [prev in thread] [next in thread] 

List:       amarok
Subject:    Fwd: extragear/multimedia/amarok/src
From:       "Dan Meltzer" <parallelgrapefruit () gmail ! com>
Date:       2008-12-15 18:38:37
Message-ID: 46059ce10812151038w13f826b0i1f5e292ed206ab7b () mail ! gmail ! com
[Download RAW message or body]

Well, Then consider this a formal request for people to test and see
if the last track in a playlist is getting its statistics updated
after this commit.


---------- Forwarded message ----------
From: Mark Kretschmann <kretschmann@kde.org>
Date: Mon, Dec 15, 2008 at 12:35 PM
Subject: Re: extragear/multimedia/amarok/src
To: kde-commits@kde.org


On Mon, Dec 15, 2008 at 5:10 PM, Dan Meltzer
<parallelgrapefruit@gmail.com> wrote:
> On Mon, Dec 15, 2008 at 4:06 AM, Mark Kretschmann <kretschmann@kde.org> wrote:
>> SVN commit 897071 by markey:
>>
>> When reaching the end of the playlist, make Amarok actually show that
>> playback has stopped. Until now it happily continued to display "Playing
>> Foo by Bar.."
>>
>> As usual with all changes to EngineController, this patch could have
>> subtle side effects, so please keep testing extensively.
>
> Without poking my head into EngineController.cpp ... Won't this change
> prevent the last tracks statistics from updating? I'm not sure how
> this change fixes the issue....

This remains to be tested. The problem was that
"m_currentTrack->finishedPlaying( 1.0 )" initiated a long chain of
things it shouldn't have been doing.

In this case, we should fix Meta::SqlTrack (or WhateverTrack)
::finishedPlaying to not trigger things that it shouldn't trigger,
rather than not callling it (and therefore not triggering things that
it should trigger)

--
Mark Kretschmann
Amarok Developer
www.kde.org - amarok.kde.org
_______________________________________________
Amarok mailing list
Amarok@kde.org
https://mail.kde.org/mailman/listinfo/amarok
[prev in list] [next in list] [prev in thread] [next in thread]