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

List:       kde-bugs-dist
Subject:    [Bug 123142] Add tempo-field to metadata
From:       Aaron VonderHaar <gruen0aermel () gmail ! com>
Date:       2006-07-11 6:44:56
Message-ID: 20060711064456.9545.qmail () ktown ! kde ! org
[Download RAW message or body]

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
         
http://bugs.kde.org/show_bug.cgi?id=123142         




------- Additional Comments From gruen0aermel gmail com  2006-07-11 08:44 -------
Created an attachment (id=16952)
 --> (http://bugs.kde.org/attachment.cgi?id=16952&action=view)
adds bpm support for mp4 files against 559637

Alf, the TagLib::MP4::Tag in amarok/src/metadata/mp4/mp4tag.* does not have
members bpm() and setBpm() which are called in metabundle.cpp in your patch... 


Were you using a custom version of taglib that included those, or did you not
try buidling with `configure ---with-mp4v2` ?


In case of the latter, here's a simple patch that should add these missing
functions.

Note however, this MP4V2_HAS_WRITE_BUG business (mp4file.cpp) is making me
nervous:  Someone who has mp4 files with BPM metadata should back up their
files and test this.  (I'll get around to it eventually...)

Applying this patch along with attachment 16921 will allow --with-mp4v2 to
succeed again.
[prev in list] [next in list] [prev in thread] [next in thread]