Review Request 109079: Patch for bug 240409 by adding bookmarks
Matěj Laitl
matej at laitl.cz
Thu Feb 21 14:05:11 UTC 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/109079/
-----------------------------------------------------------
(Updated Feb. 21, 2013, 2:05 p.m.)
Review request for Amarok.
Changes
-------
Bug should be just number.
Description
-------
This patch adds chapter support to m4b,m4a file formats. For now, libmp4v2 is a required dependency but will make it optional in the next update.
I thought we could give the user an option to choose if he wants the bookmarks or the multitrack display of chapters, but that's up for discussion.
This addresses bug 240409.
https://bugs.kde.org/show_bug.cgi?id=240409
Diffs
-----
CMakeLists.txt d96cbc9
cmake/modules/FindLibMP4.cmake PRE-CREATION
src/playlist/PlaylistController.cpp e5bde8b
Diff: http://git.reviewboard.kde.org/r/109079/diff/
Testing
-------
Thanks,
Abhinandan Ramaprasath
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20130221/b8b98937/attachment.html>
More information about the Amarok-devel
mailing list