<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/103955/">http://git.reviewboard.kde.org/r/103955/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On April 15th, 2012, 9:55 p.m., <b>Milian Wolff</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">hmm trying it out - what should I notice? could you give an example please, maybe with some example file?</pre>
</blockquote>
<p>On April 16th, 2012, 3:19 a.m., <b>Victor Vicente Carvalho</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Just hit Ctrl+alt+C to open the class quickopen menu, and try finding any class. With the path, the closest named classes will appear as the first items whereas on kdevelop without the patch the items will be ordered on lexicographical order - which is not desired. The best way to notice it is to try to match a class that has the same name as namespace, like (these classes are from a game I'm developing):
player::Ball
player::Arm
player::Crane
player::Crawler
player::Player
Looking for "player" will display the items in this exact order, whereas with the path they will be displayed as:
player::Player
player::Crane
player::Arm
player::Crawler
player::Ball
Putting the desired class, not the namespace as the preferred item.
</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">If this patch is accepted, I would like to do the same configuration for functions, as I find quite ennerving having to mannually scrolldown the quickopen to find the class constructors, as they have the same naming as the class declaration and get organized the same way.</pre>
<br />
<p>- Victor Vicente</p>
<br />
<p>On April 15th, 2012, 8:44 p.m., Victor Vicente Carvalho 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 KDevelop.</div>
<div>By Victor Vicente Carvalho.</div>
<p style="color: grey;"><i>Updated April 15, 2012, 8:44 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;">Improve QuickOpen system to sort items based on a right-to-left approach, putting items that are close to the typed text next to the first matches. This is achieved through a weighted sorter on the ProjectItemDataProvider filter.</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;">Ran kdevelop over several projects, no problems so far.</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>plugins/quickopen/projectitemquickopen.cpp <span style="color: grey">(1caadffee64051a0d77d7a65a97ff86b33a78d39)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/103955/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>