<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/124894/">https://git.reviewboard.kde.org/r/124894/</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;">OK, it's getting closer to be ready now. Good job!</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Couple of nitpicks which couldn't really fit in another comment:
* I would rename memfn.h to mem_fn.h to be more aligned with the boost naming
* Don't use preprocessor indentation like "# ifdef" we don't do that in other places, same thing for the code in the preprocessor blocks.</p></pre>
<br />
<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/124894/diff/3/?file=399362#file399362line31" style="color: black; font-weight: bold; text-decoration: underline;">src/utils/memfn.h</a>
<span style="font-weight: normal;">
(Diff revision 3)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">31</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="cp"># ifdef QSHAREDPOINTER_H</span></pre></td>
</tr>
</tbody>
</table>
<div style="margin-left: 2em;">
<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;">I'm still not in love with that, so I would say: either you manage to turn your get_pointer overload into a template where QSharedPointer doesn't appear, or you include QSharedPointer here.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I know your point about "it shall not bring extra dependency" which is true but at the same time boost/mem_fn.hpp already brings deps on the std smart pointers it's not really different.</p></pre>
</div>
</div>
<br />
<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/124894/diff/3/?file=399362#file399362line51" style="color: black; font-weight: bold; text-decoration: underline;">src/utils/memfn.h</a>
<span style="font-weight: normal;">
(Diff revision 3)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">51</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="k">auto</span> <span class="n">mem_fn</span><span class="p">(</span><span class="n">PTMorPTMF</span> <span class="k">const</span><span class="o">&</span> <span class="n">ptm</span><span class="p">)</span> <span class="o">-></span> <span class="n">decltype</span><span class="p">(</span><span class="n">boost</span><span class="o">::</span><span class="n">mem_fn</span><span class="p">(</span><span class="n">ptm</span><span class="p">))</span> <span class="p">{</span><span class="ew"> </span></pre></td>
</tr>
</tbody>
</table>
<div style="margin-left: 2em;">
<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;">Trailing whitespace</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Good call on the decltype BTW... dunno why I didn't think of it earlier. :-)</p></pre>
</div>
</div>
<br />
<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/124894/diff/3/?file=399362#file399362line55" style="color: black; font-weight: bold; text-decoration: underline;">src/utils/memfn.h</a>
<span style="font-weight: normal;">
(Diff revision 3)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">55</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="cp">#elif __cplusplus >= 201402</span></pre></td>
</tr>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">56</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
</tr>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">57</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="c1">// C++14</span></pre></td>
</tr>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">58</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="n">template</span> <span class="o"><</span><span class="kr">typename</span> <span class="n">PTMorPTMF</span><span class="o">></span></pre></td>
</tr>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">59</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="k">auto</span> <span class="n">mem_fn</span><span class="p">(</span><span class="n">PTMorPTMF</span> <span class="k">const</span><span class="o">&</span> <span class="n">ptm</span><span class="p">)</span> <span class="p">{</span><span class="ew"> </span></pre></td>
</tr>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">60</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="k">return</span> <span class="n">std</span><span class="o">::</span><span class="n">mem_fn</span><span class="p">(</span><span class="n">ptm</span><span class="p">);</span></pre></td>
</tr>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">61</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="p">}</span></pre></td>
</tr>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">62</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
</tr>
</tbody>
</table>
<div style="margin-left: 2em;">
<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;">I would say this block is not needed, it's just making the read more complex. Let's settle only on the C++11 variant.</p></pre>
</div>
</div>
<br />
<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/124894/diff/3/?file=399362#file399362line59" style="color: black; font-weight: bold; text-decoration: underline;">src/utils/memfn.h</a>
<span style="font-weight: normal;">
(Diff revision 3)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">59</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="k">auto</span> <span class="n">mem_fn</span><span class="p">(</span><span class="n">PTMorPTMF</span> <span class="k">const</span><span class="o">&</span> <span class="n">ptm</span><span class="p">)</span> <span class="p">{</span><span class="ew"> </span></pre></td>
</tr>
</tbody>
</table>
<div style="margin-left: 2em;">
<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;">Trailing whitespace</p></pre>
</div>
</div>
<br />
<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/124894/diff/3/?file=399362#file399362line66" style="color: black; font-weight: bold; text-decoration: underline;">src/utils/memfn.h</a>
<span style="font-weight: normal;">
(Diff revision 3)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">66</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="k">auto</span> <span class="n">mem_fn</span><span class="p">(</span><span class="n">PTMorPTMF</span> <span class="k">const</span><span class="o">&</span> <span class="n">ptm</span><span class="p">)</span> <span class="o">-></span> <span class="n">decltype</span><span class="p">(</span><span class="n">std</span><span class="o">::</span><span class="n">mem_fn</span><span class="p">(</span><span class="n">ptm</span><span class="p">))</span> <span class="p">{</span><span class="ew"> </span></pre></td>
</tr>
</tbody>
</table>
<div style="margin-left: 2em;">
<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;">Trailing whitespace</p></pre>
</div>
</div>
<br />
<p>- Kevin Ottens</p>
<br />
<p>On August 27th, 2015, 9:07 p.m. UTC, René J.V. Bertin 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 KDE Software on Mac OS X and Zanshin.</div>
<div>By René J.V. Bertin.</div>
<p style="color: grey;"><i>Updated Aug. 27, 2015, 9:07 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
zanshin
</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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This is a set of patches that allow zanshin version <code style="text-rendering: inherit;color: #4444cc;padding: 0;white-space: normal;margin: 0;line-height: inherit;">0.2.1-1163-gbf5c321</code> to build on OS X against the git head (or almost head) of kdelibs 4.14 and kdepim* 4.14 .
I presume for now that a number of changes are self-explanatory (for instance, <code style="text-rendering: inherit;color: #4444cc;padding: 0;white-space: normal;margin: 0;line-height: inherit;">zanshinkdepimstatic</code> calls itself static, isn't installed and doesn't export any symbols explicitly so all points to it being intended to be a static instead of a shared lib).
The patches that disable a number of tests, and I do not understand myself what exactly goes wrong without them other than that the linker gives errors like</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;"><div class="codehilite" style="background: #f8f8f8"><pre style="line-height: 125%"><span style="border: 1px solid #FF0000">#</span> Undefined symbols <span style="color: #008000; font-weight: bold">for</span> architecture x86_64<span style="color: #666666">:</span>
<span style="border: 1px solid #FF0000">#</span> <span style="color: #BA2121">"decltype(*(std::__1::forward<QSharedPointer<Domain::QueryResult<QString> > >(fp0)).*fp(std::__1::forward<>(fp1))) std::__1::__invoke<QList<std::__1::function<void (QString, int)> > (Domain::QueryResult<QString>::*&)() const, QSharedPointer<Domain::QueryResult<QString> >, void>(QList<std::__1::function<void (QString, int)> > (Domain::QueryResult<QString>::*&&&)() const, QSharedPointer<Domain::QueryResult<QString> >&&)"</span>, referenced from<span style="color: #666666">:</span>
<span style="border: 1px solid #FF0000">#</span> std<span style="color: #666666">::</span>__1<span style="color: #666666">::</span>__function<span style="color: #666666">::</span>__func<span style="color: #666666"><</span>std<span style="color: #666666">::</span>__1<span style="color: #666666">::</span>__mem_fn<span style="color: #666666"><</span>QList<span style="color: #666666"><</span>std<span style="color: #666666">::</span>__1<span style="color: #666666">::</span><span style="color: #008000; font-weight: bold">function</span><span style="color: #666666"><</span><span style="color: #008000; font-weight: bold">void</span> (QString, <span style="color: #008000; font-weight: bold">int</span>)<span style="color: #666666">></span> <span style="color: #666666">></span> (Domain<span style="color: #666666">::</span>QueryResult<span style="color: #666666"><</span>QString<span style="color: #666666">>::*</span>)() <span style="color: #008000; font-weight: bold">const</span><span style="color: #666666">></span>, std<span style="color: #666666">::</span>__1<span style="color: #666666">::</span>allocator<span style="color: #666666"><</span>std<span style="color: #666666">::</span>__1<span style="color: #666666">::</span>__mem_fn<span style="color: #666666"><</span>QList<span style="color: #666666"><</span>std<span style="color: #666666">::</span>__1<span style="color: #666666">::</span><span style="color: #008000; font-weight: bold">function</span><span style="color: #666666"><</span><span style="color: #008000; font-weight: bold">void</span> (QString, <span style="color: #008000; font-weight: bold">int</span>)<span style="color: #666666">></span> <span style="color: #666666">></span> (Domain<span style="color: #666666">::</span>QueryResult<span style="color: #666666"><</span>QString<span style="color: #666666">>::*</span>)() <span style="color: #008000; font-weight: bold">const</span><span style="color: #666666">></span> <span style="color: #666666">></span>, QList<span style="color: #666666"><</span>std<span style="color: #666666">::</span>__1<span style="color: #666666">::</span><span style="color: #008000; font-weight: bold">function</span><span style="color: #666666"><</span><span style="color: #008000; font-weight: bold">void</span> (QString, <span style="color: #008000; font-weight: bold">int</span>)<span style="color: #666666">></span> <span style="color: #666666">></span> (QSharedPointer<span style="color: #666666"><</span>Domain<span style="color: #666666">::</span>QueryResult<span style="color: #666666"><</span>QString<span style="color: #666666">></span> <span style="color: #666666">></span>)<span style="color: #666666">>::</span>operator()(QSharedPointer<span style="color: #666666"><</span>Domain<span style="color: #666666">::</span>QueryResult<span style="color: #666666"><</span>QString<span style="color: #666666">></span> <span style="color: #666666">>&&</span>) <span style="color: #008000; font-weight: bold">in</span> queryresulttest.o
<span style="border: 1px solid #FF0000">#</span> ld<span style="color: #666666">:</span> symbol(s) not found <span style="color: #008000; font-weight: bold">for</span> architecture x86_64
<span style="border: 1px solid #FF0000">#</span> clang<span style="color: #666666">:</span> error<span style="color: #666666">:</span> linker command failed <span style="color: #008000; font-weight: bold">with</span> exit code <span style="color: #666666">1</span> (use <span style="color: #666666">-</span>v to see invocation)
</pre></div>
</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Possibly a comparable to the reason why I had to define the <code style="text-rendering: inherit;color: #4444cc;padding: 0;white-space: normal;margin: 0;line-height: inherit;">std::accumulate</code> functions.</p></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;">On OS X 10.9.5 with the system compiler (<code style="text-rendering: inherit;color: #4444cc;padding: 0;white-space: normal;margin: 0;line-height: inherit;">Apple LLVM version 6.0 (clang-600.0.57) (based on LLVM 3.5svn)</code>)</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">My initial reason to install this was to "check it out" and learn how to use it, so I have not done any further testing ...</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>3rdparty/kdepim/libkdepim/CMakeLists.txt <span style="color: grey">(cc8845a)</span></li>
<li>3rdparty/kdepim/libkdepim/tests/CMakeLists.txt <span style="color: grey">(775ac0e)</span></li>
<li>CMakeLists.txt <span style="color: grey">(71b16a8)</span></li>
<li>src/akonadi/akonadiserializer.cpp <span style="color: grey">(5116fa5)</span></li>
<li>src/domain/queryresultprovider.h <span style="color: grey">(5d0b02d)</span></li>
<li>src/presentation/errorhandler.cpp <span style="color: grey">(5610040)</span></li>
<li>src/utils/memfn.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>tests/testlib/akonadistoragetestbase.cpp <span style="color: grey">(8573374)</span></li>
<li>tests/units/akonadi/akonadidatasourcequeriestest.cpp <span style="color: grey">(c2e87dc)</span></li>
<li>tests/units/widgets/applicationcomponentstest.cpp <span style="color: grey">(f972673)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/124894/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">File Attachments </h1>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2015/08/24/b2f3e229-ca5a-4e55-b33b-6504b2cdb5c7__datasourcequeriestest-AppleClang6.i">datasourcequeriestest.cpp preprocessed by Apple Clang 6</a></li>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2015/08/24/807e30a4-aa34-45b4-9e5c-92020ee8c7f5__datasourcequeriestestClang3.6.i">datasourcequeriestest.cpp preprocessed by clang 3.6.1</a></li>
</ul>
</td>
</tr>
</table>
</div>
</body>
</html>