<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/112626/">http://git.reviewboard.kde.org/r/112626/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://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 Digikam, Gilles Caulier and Marcel Wiesweg.</div>
<div>By Markus Leuthold.</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;">Improve tag prediciton for taglineedit
Tag prediction for the taglineedit-widget, introduced in d13f94a8c5cf18, relies on the order of tag usage. A more recently used tag will show up first in the taglineedit dropdown compared to a tag which was assigned long time ago. For this purpose, the recentlyAssignedTags list from the AlbumDB class can be used. The length of the list is currently limited to 10 entries, which is way too short for a reasonable tag prediction. Increasing recentlyUsedTags buffer from 10 to 200 gives quite good result. Only the first 10 tags are displayed, but internally, the first 200 are saved.
The behavior of recently used tags in digikam is not affected, additional memory usage on the disk is minimal (200 * 4 bytes in digikamrc)
</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;">I'm successfully using this feature since April</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>digikam/tags/tagspopupmenu.cpp <span style="color: grey">(c6154b1dd5f2a544a323c484dfea38d35ca1352f)</span></li>
<li>libs/database/albumdb.cpp <span style="color: grey">(8ca4fd0e0323fb7c8bae65f947d21b9d9ee6b50c)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/112626/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>