<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/112815/">http://git.reviewboard.kde.org/r/112815/</a>
</td>
</tr>
</table>
<br />
<p>On September 23rd, 2013, 10:58 a.m. UTC, <b>Myriam Schweingruber</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I am just wondering: since Amarok doesn't play APE tracks anyway, why bother? AFAIK there is no phonon backend currently that provides the MAC codec, and ape files shouldn't even show up in the collection folder, as neither the vlc nor gstreamer backends do support playing these files.</pre>
</blockquote>
<p>On September 23rd, 2013, 12:32 p.m. UTC, <b>Bruno Léon</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">The issue title seems to be misleading but this is about APE tags (APEv2) format not APE audio files.
In my case this is for Musepack which a lot of my collection is using.</pre>
</blockquote>
<p>On September 23rd, 2013, 3:34 p.m. UTC, <b>Myriam Schweingruber</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">And which phonon backend and codec do you use to play them? I have not seen any phonon backend supporting them, and there is no codec available for gstramer or vlc.
I know you can play musepack with mpc123 or the like, but it doesn't play in Amarok, so what is the point?</pre>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Musepack files play fine both with gstreamer or vlc backend for sure. I've been playing them for years.</pre>
<br />
<p>- Bruno</p>
<br />
<p>On September 19th, 2013, 1:12 p.m. UTC, Bruno Léon wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Amarok.</div>
<div>By Bruno Léon.</div>
<p style="color: grey;"><i>Updated Sept. 19, 2013, 1:12 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Properly fix read compilation tag in APE (musepack...) files
Previous patch 5e5140b was not complete. The information was read but not inserted in the data structure and thus not inserted in Aamrok DB.
</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Files that were not placed in Various Artists before are now correctly recognised as compilation.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>shared/tag_helpers/APETagHelper.cpp <span style="color: grey">(ba39a10)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/112815/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>