<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/126682/">https://git.reviewboard.kde.org/r/126682/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Hi. As you can see there is more than one version of decoder. I had to add separate code blocks for different versions of ffmpeg because of weak backward capability from ffmpeg side (call of old API methods caused crushes while It sill compiled well). So I think that It would be better to add one more conditional compilation section for new version (version range) of ffmpeg. But if you pretty sure that your version will compile and work with ffmpeg 0.11 It's ok to keep It like this. Btw probably It's time to drop support of antiant versions like 0.7 and 0.8.</p></pre>
<br />
<p>- Sergey Ivanov</p>
<br />
<p>On Январь 9th, 2016, 6:24 д.п. UTC, Diane Trout wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for Amarok.</div>
<div>By Diane Trout.</div>
<p style="color: grey;"><i>Updated Янв. 9, 2016, 6:24 д.п.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
amarok
</div>
<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;">See API change documentation:
https://anonscm.debian.org/cgit/collab-maint/ffmpeg.git/tree/doc/APIchanges
https://wiki.libav.org/Migration/12</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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I installed the patch on the debian 2.8.0 branch and it build successfully. I then tried to trigger the affected code, however it crashed because of https://bugs.kde.org/show_bug.cgi?id=328359</p></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>src/musicbrainz/MusicDNSAudioDecoder.cpp <span style="color: grey">(6216e5ce26cc3461b89aea560943ad7a79101af9)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/126682/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>