<br><br><div class="gmail_quote">On Thu, Feb 21, 2013 at 4:17 AM, Matěj Laitl <span dir="ltr"><<a href="mailto:matej@laitl.cz" target="_blank">matej@laitl.cz</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On 19. 2. 2013 Abhinandan Ramprasath wrote:<br>
</div><div class="im">> This is what I've got so far,<br>
<br>
</div>Nice research!<br>
<div class="im"><br>
> 1) mp4v2-2.0.0 supports chapters in mp4 files(mentioned on comment 2). The<br>
> format for chapter storage is specified between lines 2321 and 2375 under<br>
> src/mp4file.cpp (<br>
> <a href="http://code.google.com/p/mp4v2/downloads/detail?name=mp4v2-2.0.0.tar.bz2" target="_blank">http://code.google.com/p/mp4v2/downloads/detail?name=mp4v2-2.0.0.tar.bz2</a> ) .<br>
<br>
</div>Okay.<br>
<div class="im"><br>
> 2) comment 2 in <a href="https://bugs.kde.org/show_bug.cgi?id=240409" target="_blank">https://bugs.kde.org/show_bug.cgi?id=240409</a> mentions that<br>
> the dependency was removed. Is it safe for me to add it back?<br>
<br>
</div>We'd prefer not to, certainly not as a mandatory dependency.<br>
<div class="im"><br>
> 3) I cross checked the format of chapter storage with a simple cpp program<br>
> I wrote and got the titles of all chapters as output. But that was just<br>
> using a regex and I'm not too sure that would be accepted as a solution.<br>
<br>
</div>RegExes are used all over Amarok, so if this is not a moving target (e.g. if<br>
the strings you match don't change with library versions), it should be fine,<br>
but see below.<br></blockquote><div>I'm not sure if it is a moving target. Will read about it and get back.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><br>
> 4) An Issue was filed for TagLib<br>
> <a href="https://github.com/taglib/taglib/issues/81" target="_blank">https://github.com/taglib/taglib/issues/81</a> asking<br>
> to expand support for chapters in mp3 files, which sadly, hasn't been<br>
> solved yet. Unfortunately, I wasn't able to find anything similar for mp4.<br>
<br>
</div>You fan file one, then! ;) Even better would be creating a pull request for<br>
TagLib that would add support for reading chapters (ideally both from mp3 and<br>
mp4 files). Given that mp4v2 is under MPL 1.1 and TagLib is under MPL 1.1 +<br>
LGPL 2.1, it may be possible to copy & adapt code from mp4v2 to TagLib<br>
directly, but please consult this first with TagLib developers [1]. (if both<br>
mp4v2 and TagLib where updated to the newest MPL 2.0, it would be definitely<br>
possible)<br></blockquote><div><br></div><div>Ya, trying to get the feature working on Taglib was the next thing I was going to do. I will try to find more information about it and get back.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
Let's defer the decision how to represent this in Amarok until underlying<br>
libraries are ready, I must say that the idea to use MultiTrack may end up to<br>
be the good way, but please let us think about this more.<br></blockquote><div>I think Multitrack is how QuickTime does it( or I have a really old version ) and it looks pretty good. </div><div><br></div><div>P.S sorry for submitting the patch, I was almost done with it when I read the email</div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
[1] <a href="https://mail.kde.org/mailman/listinfo/taglib-devel" target="_blank">https://mail.kde.org/mailman/listinfo/taglib-devel</a><br>
<span class="HOEnZb"><font color="#888888"><br>
        Matěj<br>
</font></span></blockquote></div><br>