<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/123563/">https://git.reviewboard.kde.org/r/123563/</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;">nice, I can see people who'd use this (not me though, I'd rather ask the VDG to improve our icons :P).</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">there are some style issues with the code, could you fix this?</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/123563/diff/1/?file=364481#file364481line27" style="color: black; font-weight: bold; text-decoration: underline;">language/duchain/duchainutils.h</a>
<span style="font-weight: normal;">
(Diff revision 1)
</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">27</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="cp">#include <QHash></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;">move to .cpp</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/123563/diff/1/?file=364482#file364482line51" style="color: black; font-weight: bold; text-decoration: underline;">language/duchain/duchainutils.cpp</a>
<span style="font-weight: normal;">
(Diff revision 1)
</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="cp">#include <foregroundlock.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;">obsolete?</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/123563/diff/1/?file=364482#file364482line104" style="color: black; font-weight: bold; text-decoration: underline;">language/duchain/duchainutils.cpp</a>
<span style="font-weight: normal;">
(Diff revision 1)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#e9eaa8" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2">95</font></th>
<td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="k">if</span> <span class="p">(</span><span class="n">dec</span><span class="o">-></span><span class="n">abstractType</span><span class="p">())</span> <span class="p">{</span></pre></td>
<th bgcolor="#e9eaa8" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">58</font></th>
<td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="k">class</span> <span class="nc">IconCache</span> <span class="p">{</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'd prefer if this would be moved into its own .h/.cpp files. duchainutils is large enough already</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/123563/diff/1/?file=364482#file364482line107" style="color: black; font-weight: bold; text-decoration: underline;">language/duchain/duchainutils.cpp</a>
<span style="font-weight: normal;">
(Diff revision 1)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#e9eaa8" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2">98</font></th>
<td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="n">p</span> <span class="o">|=</span> <span class="n">CodeCompletionModel</span><span class="o">::</span><span class="n">Variable</span><span class="p">;</span></pre></td>
<th bgcolor="#e9eaa8" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">61</font></th>
<td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="n">ICompletionSettings</span><span class="o">::</span><span class="n">IconStyle</span> <span class="n">iconStyle</span><span class="p">();</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;">this can be made private, no?</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/123563/diff/1/?file=364482#file364482line111" style="color: black; font-weight: bold; text-decoration: underline;">language/duchain/duchainutils.cpp</a>
<span style="font-weight: normal;">
(Diff revision 1)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#e9eaa8" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2">102</font></th>
<td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="k">break</span><span class="p">;</span></pre></td>
<th bgcolor="#e9eaa8" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">65</font></th>
<td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="n">QPixmap</span> <span class="nf">iconWithVisibilityOverlay</span><span class="p">(</span><span class="n">QString</span> <span class="n">name</span><span class="p">,</span> <span class="n">CodeCompletionModel</span><span class="o">::</span><span class="n">CompletionProperties</span> <span class="n">p</span><span class="p">);</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;">document:</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">// only eclipse icons</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/123563/diff/1/?file=364482#file364482line153" style="color: black; font-weight: bold; text-decoration: underline;">language/duchain/duchainutils.cpp</a>
<span style="font-weight: normal;">
(Diff revision 1)
</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">95</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">clearCache</span> <span class="o">=</span> <span class="p">[</span><span class="k">this</span><span class="p">]()</span> <span class="p">{</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;">this should be in the ctor of the IconCache, no?</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">generally, I'd expect this function to work as a free function with the contents</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">ICompletionSettings::IconStyle iconStyle()
{
return ICore::self()->languageController()->completionSettings();
}</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">when can the settings be zero?</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/123563/diff/1/?file=364482#file364482line256" style="color: black; font-weight: bold; text-decoration: underline;">language/duchain/duchainutils.cpp</a>
<span style="font-weight: normal;">
(Diff revision 1)
</span>
</th>
</tr>
</thead>
<tbody style="background-color: #e4d9cb; padding: 4px 8px; text-align: center;">
<tr>
<td colspan="2"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">QIcon DUChainUtils::iconForDeclaration(const Declaration* dec)</pre></td>
<td colspan="2"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">QPixmap IconCache::iconWithVisibilityOverlay(QString name, CodeCompletionModel::CompletionProperties p) {</pre></td>
</tr>
</tbody>
<tbody>
<tr>
<th bgcolor="#e9eaa8" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2">234</font></th>
<td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="n">QIcon</span> <span class="n">DUChainUtils</span><span class="o">::</span><span class="n">iconForDeclaration</span><span class="p">(</span><span class="k">const</span> <span class="n">Declaration</span><span class="o">*</span> <span class="n">dec</span><span class="p">)</span></pre></td>
<th bgcolor="#e9eaa8" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">196</font></th>
<td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="n">QPixmap</span> <span class="n">IconCache</span><span class="o">::</span><span class="n">iconWithVisibilityOverlay</span><span class="p">(</span><span class="n">QString</span> <span class="n">name</span><span class="p">,</span> <span class="n">CodeCompletionModel</span><span class="o">::</span><span class="n">CompletionProperties</span> <span class="n">p</span><span class="p">)</span> <span class="p">{</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;">here and elsewhere: put { on its own line</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/123563/diff/1/?file=364482#file364482line258" style="color: black; font-weight: bold; text-decoration: underline;">language/duchain/duchainutils.cpp</a>
<span style="font-weight: normal;">
(Diff revision 1)
</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">198</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="n">qCWarning</span><span class="p">(</span><span class="n">LANGUAGE</span><span class="p">)</span> <span class="o"><<</span> <span class="s">"Forward declared: "</span> <span class="o"><<</span> <span class="n">p</span> <span class="o"><<</span> <span class="n">name</span><span class="p">;</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;">warning? why?</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/123563/diff/1/?file=364482#file364482line268" style="color: black; font-weight: bold; text-decoration: underline;">language/duchain/duchainutils.cpp</a>
<span style="font-weight: normal;">
(Diff revision 1)
</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">208</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="p">}</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;">join line, also below</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/123563/diff/1/?file=364482#file364482line425" style="color: black; font-weight: bold; text-decoration: underline;">language/duchain/duchainutils.cpp</a>
<span style="font-weight: normal;">
(Diff revision 1)
</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">365</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="k">static</span> <span class="n">CodeCompletionModel</span><span class="o">::</span><span class="n">CompletionProperties</span> <span class="n">completionPropertiesImpl</span><span class="p">(</span><span class="k">const</span> <span class="n">Declaration</span><span class="o">*</span> <span class="n">dec</span><span class="p">)</span> <span class="p">{</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;">you are in an anon namespace, no? so the static is obsolete</p></pre>
</div>
</div>
<br />
<p>- Milian Wolff</p>
<br />
<p>On April 29th, 2015, 12:56 p.m. UTC, Alex Richardson 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 KDevelop.</div>
<div>By Alex Richardson.</div>
<p style="color: grey;"><i>Updated April 29, 2015, 12:56 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kdevplatform
</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;">I personally find them a lot more intuitive than the Visual Studio ones that
KDevelop ships (C = class, U = Union, S = Struct, T = typedef and N = namespace).
I still had an old branch that replaced the KDevelop icons from a few years ago.
I now cleaned it up and added them as an additional option besides the existing
icon set. It is configurable at runtime but might take a while until all currently
displayed icons have been updated.</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;">Icons are loaded, switching between themes also works</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Examples below are from the outline view plugin (https://git.reviewboard.kde.org/r/123562/)</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>interfaces/icompletionsettings.h <span style="color: grey">(4b95c5184cf3b950907f291986a1efa2ad48d032)</span></li>
<li>language/duchain/duchainutils.h <span style="color: grey">(7d4760af955f37c0ad87f4635ddaa8ce31eb445b)</span></li>
<li>language/duchain/duchainutils.cpp <span style="color: grey">(8a26194dcadeb98c94785fecd1b69c91ca0b4502)</span></li>
<li>pics/CMakeLists.txt <span style="color: grey">(6e046c7fcb5eb29a6c3331c13cdbeff992946a8a)</span></li>
<li>pics/codecompletion/CMakeLists.txt <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_class.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_class_forward.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_class_package.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_class_private.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_class_protected.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_class_public.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_define.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_enum.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_enum_forward.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_enumerator.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_field_package.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_field_private.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_field_protected.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_field_public.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_function_global.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_function_package.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_function_private.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_function_protected.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_function_public.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_global_function_declaration.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_implement.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_interface.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_interface_forward.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_namespace.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_overlay_abstract.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_overlay_const.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_overlay_implement.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_overlay_override.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_overlay_package_access.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_overlay_private.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_overlay_protected.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_overlay_public.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_overlay_static.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_overlay_virtual.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_override.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_shadows.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_struct.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_struct_forward.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_type_variable.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_typedef.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_typedef_forward.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_union.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_union_forward.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_unknown_type.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_unknown_type_2.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_using.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_variable_declaration.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_variable_global.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_variable_local.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>pics/codecompletion/eclipse_variable_type.png <span style="color: grey">(PRE-CREATION)</span></li>
<li>shell/completionsettings.h <span style="color: grey">(05f166b4323c773c38ceb6703f36aabe35900636)</span></li>
<li>shell/completionsettings.cpp <span style="color: grey">(89cf68b46e312e0a54cd67cf7235ff5930833f82)</span></li>
<li>shell/settings/ccconfig.kcfg <span style="color: grey">(624d8da97580266f52220258e7aa42e0067e76a3)</span></li>
<li>shell/settings/ccpreferences.cpp <span style="color: grey">(9346e2397ea4225f9bde02ab4c30889ab0732d24)</span></li>
<li>shell/settings/ccpreferences.ui <span style="color: grey">(48bd90f44f670b5ea879bc6b187ead8c9bcbba30)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/123563/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/04/29/91b275a1-006e-4752-a322-2404cf674595__eclipse_icons1.png">eclipse_icons1.png</a></li>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2015/04/29/d365bd2b-20a5-4b8f-9743-84efbf552577__eclipse_icons2.png">eclipse_icons2.png</a></li>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2015/04/29/ef3ecddd-2a0e-40d1-acd4-db105d2e3748__eclipse_icons3.png">eclipse_icons3.png</a></li>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2015/04/29/b22651f8-f260-467a-85b6-36592f7176fd__eclipse_icons4.png">eclipse_icons4.png</a></li>
</ul>
</td>
</tr>
</table>
</div>
</body>
</html>