<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="https://git.reviewboard.kde.org/r/117954/">https://git.reviewboard.kde.org/r/117954/</a>
</td>
</tr>
</table>
<br />
<p>Ship it!</p>
<div>
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
<thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
<a href="https://git.reviewboard.kde.org/r/117954/diff/2/?file=273791#file273791line122" style="color: black; font-weight: bold; text-decoration: underline;">keditfiletype/mimetypedata.h</a>
<span style="font-weight: normal;">
(Diff revision 2)
</span>
</th>
</tr>
</thead>
<tbody style="background-color: #e4d9cb; padding: 4px 8px; text-align: center;">
<tr>
<td colspan="4"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">public:</pre></td>
</tr>
</tbody>
<tbody>
<tr>
<th bgcolor="#e9eaa8" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2">119</font></th>
<td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="n"><span class="hl">K</span>MimeType</span><span class="o"><span class="hl">::</span></span><span class="n"><span class="hl">Ptr</span></span> <span class="n">m_mimetype</span><span class="p">;</span> <span class="c1">// 0 if this is data for a mimetype group (m_isGroup==true)</span></pre></td>
<th bgcolor="#e9eaa8" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">122</font></th>
<td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="n"><span class="hl">Q</span>MimeType</span> <span class="n">m_mimetype</span><span class="p">;</span> <span class="c1">// 0 if this is data for a mimetype group (m_isGroup==true)</span></pre></td>
</tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">comment is wrong, can't be 0 anymore.</pre>
</div>
<br />
<p>- David Faure</p>
<br />
<p>On May 23rd, 2014, 8:45 a.m. UTC, Aurélien Gâteau wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://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 Plasma and David Faure.</div>
<div>By Aurélien Gâteau.</div>
<p style="color: grey;"><i>Updated May 23, 2014, 8:45 a.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kde-cli-tools
</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;">This patch contains multiple commits, which I can make available if needed. It ports keditfiletype away from kde4support, which helps identifying its dependencies so that it can hopefully be moved closer to the classes which need it in frameworks (right now kwidgetsaddons and kio forks it)</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;">Program starts and looks like it behaves correctly. Unit-test is failing, but was already failing before :/</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>CMakeLists.txt <span style="color: grey">(d62ee8b)</span></li>
<li>keditfiletype/CMakeLists.txt <span style="color: grey">(566e76a)</span></li>
<li>keditfiletype/filegroupdetails.cpp <span style="color: grey">(c733ba4)</span></li>
<li>keditfiletype/filetypedetails.h <span style="color: grey">(7c0abb4)</span></li>
<li>keditfiletype/filetypedetails.cpp <span style="color: grey">(bfcbb07)</span></li>
<li>keditfiletype/filetypesview.h <span style="color: grey">(82fd0ab)</span></li>
<li>keditfiletype/filetypesview.cpp <span style="color: grey">(886892f)</span></li>
<li>keditfiletype/keditfiletype.h <span style="color: grey">(0b235ce)</span></li>
<li>keditfiletype/keditfiletype.cpp <span style="color: grey">(aab0830)</span></li>
<li>keditfiletype/kservicelistwidget.h <span style="color: grey">(c817d24)</span></li>
<li>keditfiletype/kservicelistwidget.cpp <span style="color: grey">(88dface)</span></li>
<li>keditfiletype/kserviceselectdlg.h <span style="color: grey">(8e6d0d0)</span></li>
<li>keditfiletype/kserviceselectdlg.cpp <span style="color: grey">(09c7c5d)</span></li>
<li>keditfiletype/mimetypedata.h <span style="color: grey">(042b043)</span></li>
<li>keditfiletype/mimetypedata.cpp <span style="color: grey">(057113f)</span></li>
<li>keditfiletype/mimetypewriter.h <span style="color: grey">(5a618a6)</span></li>
<li>keditfiletype/mimetypewriter.cpp <span style="color: grey">(be5e5d4)</span></li>
<li>keditfiletype/newtypedlg.h <span style="color: grey">(fba8385)</span></li>
<li>keditfiletype/newtypedlg.cpp <span style="color: grey">(c933a8f)</span></li>
<li>keditfiletype/sharedmimeinfoversion.h <span style="color: grey">(9824d7b)</span></li>
<li>keditfiletype/sharedmimeinfoversion.cpp <span style="color: grey">(9dabf0e)</span></li>
<li>keditfiletype/tests/CMakeLists.txt <span style="color: grey">(489bb4d)</span></li>
<li>keditfiletype/tests/filetypestest.cpp <span style="color: grey">(68eef26)</span></li>
<li>keditfiletype/typeslistitem.h <span style="color: grey">(05e425a)</span></li>
<li>keditfiletype/typeslistitem.cpp <span style="color: grey">(97cd3c1)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/117954/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>