Review Request: Fix CUE file track seeking

[Matěj Laitl]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107193/#review21406
-----------------------------------------------------------


Hmm, yes, we'll have to implement something like this to fix the problem. On the other hand, I'd like to implement it using a RAI principle as shown in https://git.reviewboard.kde.org/r/105995/ - that patch now just waits for phonon-vlc 0.6.1 to be released. When it is merged, equivalent fix could be applied using DelayedSeeker.

- Matěj Laitl


On Nov. 4, 2012, 10:08 a.m., Ben Cooksley wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/107193/
> -----------------------------------------------------------
> 
> (Updated Nov. 4, 2012, 10:08 a.m.)
> 
> 
> Review request for Amarok.
> 
> 
> Description
> -------
> 
> This (for the most part) fixes the automatic seeking specified by CUE files as one changes tracks, with Phonon GStreamer at least.
> 
> It seems that GStreamer refuses to perform any form of seeking until the file has commenced playback, so we must initiate playback before we can seek.
> Unfortunately, due to the async nature of Phonon::MediaObject->play() we must use the PlayingState signal to perform the seeking.
> 
> Not sure if this should be integrated due to that, or if there are better ways to refine it (maybe asking it to play and then immediately pausing it, seeking and then resuming playback - although that could cause a stutter instead). 
> 
> 
> Diffs
> -----
> 
>   src/EngineController.cpp 955e049 
> 
> Diff: http://git.reviewboard.kde.org/r/107193/diff/
> 
> 
> Testing
> -------
> 
> Tested locally using Phonon GStreamer. CUE Files now cause tracks to be seeked to appropriately as Amarok moves between tracks.
> There is a tiny amount (a few ms worth) which is played from the start of the track before the seeking kicks in unfortunately.
> 
> 
> Thanks,
> 
> Ben Cooksley
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20121104/0a47e432/attachment.html>