Review Request: Cue sheet not working for files in collection
Bart Cerneels
bart.cerneels at kde.org
Wed Jan 5 09:45:28 CET 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/100301/#review735
-----------------------------------------------------------
This is clearly not the way to implement Cue support in SqlCollection.
The scanner needs to detect cue/mp3 pairs and the collection needs to create a track for each cue entry using timecode bounds.
But we already knew that, just haven't implemented it yet.
Anton: feel free to keep using your patch, but we probably will not merge it.
Bart
- Bart
On 2011-01-04 19:15:07, Anton Revunov wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/100301/
> -----------------------------------------------------------
>
> (Updated 2011-01-04 19:15:07)
>
>
> Review request for Amarok.
>
>
> Summary
> -------
>
> Cue sheet not working for files in collection
>
>
> This addresses bug 187587.
> https://bugs.kde.org/show_bug.cgi?id=187587
>
>
> Diffs
> -----
>
> src/playlist/PlaylistController.h 421274d
> src/playlist/PlaylistController.cpp d38c9d6
>
> Diff: http://git.reviewboard.kde.org/r/100301/diff
>
>
> Testing
> -------
>
> I made a patch for Amarok 2.3.2 as a temporary solution. It works for me and my collection. Required to re-add tracks to the playlist. Does not work for drag-and-drop and maybe for something else.
>
>
> Thanks,
>
> Anton
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/amarok-devel/attachments/20110105/01cda523/attachment-0001.htm
More information about the Amarok-devel
mailing list