<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/106967/">http://git.reviewboard.kde.org/r/106967/</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;">Over a week and still no comments. If no one objects in the next few days, I'm simply gonna push this without any "Ship It!".</pre>
<br />
<p>- Jon</p>
<br />
<p>On October 20th, 2012, 6:25 p.m., Jon Severinsson wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for KDE Frameworks and Phonon.</div>
<div>By Jon Severinsson.</div>
<p style="color: grey;"><i>Updated Oct. 20, 2012, 6:25 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;">This patch is a cleanup patch that makes the "five" branch compile after merging master into five.
The merge this patch is based on includes my two patches "Port from QAbstractItemModel::reset() to QAbstractItemModel::(begin|end)ResetModel()." and "Fix build without OpenGL.", but the patch should work without them if they get rejected.
This is the first step in getting Phonon five running on Qt5, so the KDE_NO_PHONON hackery can be dropped from the kdelibs framework branch, but at this point Phonon five will still only work on Qt4, further patches are comming :)</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>Mainpage.dox <span style="color: grey">(82b96ad)</span></li>
<li>PhononConfig.cmake.in <span style="color: grey">(4e46016)</span></li>
<li>doc/CMakeLists.txt <span style="color: grey">(24476d6)</span></li>
<li>includes/CMakeLists.txt <span style="color: grey">(7f88f08)</span></li>
<li>includes/old/Phonon/Experimental/AbstractVideoDataOutput <span style="color: grey">(c019215)</span></li>
<li>includes/old/Phonon/Experimental/AudioDataOutput <span style="color: grey">(06d181d)</span></li>
<li>includes/old/Phonon/Experimental/SnapshotInterface <span style="color: grey">(5e59306)</span></li>
<li>includes/old/Phonon/Experimental/VideoDataOutput <span style="color: grey">(9878ccf)</span></li>
<li>includes/old/Phonon/Experimental/VideoDataOutputInterface <span style="color: grey">(870f683)</span></li>
<li>includes/old/Phonon/Experimental/VideoFrame <span style="color: grey">(97c137e)</span></li>
<li>includes/old/Phonon/Experimental/VideoFrame2 <span style="color: grey">(29d1af9)</span></li>
<li>includes/old/Phonon/Experimental/Visualization <span style="color: grey">(b5aab38)</span></li>
<li>includes/phonon/AvCapture <span style="color: grey">(15e92c5)</span></li>
<li>lib-descriptor.xml.cmake <span style="color: grey">(fe8a0d6)</span></li>
<li>phonon/CMakeLists.txt <span style="color: grey">(d0f6dde)</span></li>
<li>phonon/abstractvideooutput.h <span style="color: grey">(b44ceb2)</span></li>
<li>phonon/backendcapabilities.h <span style="color: grey">(0e8f7ec)</span></li>
<li>phonon/phononnamespace.cpp <span style="color: grey">(88e38e0)</span></li>
<li>phonon/phononnamespace.h.in <span style="color: grey">(08a33c2)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/106967/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>