<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/100398/">http://git.reviewboard.kde.org/r/100398/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On January 17th, 2011, 11:25 a.m., <b>Andrea Diamantini</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;">uhm... it seems faster, you're right. What I could not properly understand is why you are actually NOT saving a ResponseList and let rekonq parse it everytime. Am I missing something?</pre>
</blockquote>
<p>On January 17th, 2011, 3:40 p.m., <b>Lionel Chauvin</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;">A Response is a structure. For save this structure, I must use a format. What format is better than the original format ?</pre>
</blockquote>
<p>On January 21st, 2011, 12:25 a.m., <b>Andrea Diamantini</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;">I see what you're saying. But I'm thinking about this: we have different original formats, but just one we use for responses. Moreover, having cached responses just parsed save one operation more (the parsing). Isn't this the heavier operation here?</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;">"we have different original formats, but just one we use for responses"
no
We parse json or xml files and we save needed data in a list of structures (ResponseList).
We don't have code for save a ResponseList in xml or json. This is why I save the dowloaded files to disk (For being parsed again later).
I think it is not interesting to create a new format for save and load a ResponseList.
"Isn't this the heavier operation here?"
heavier ? CPU usage or time ?
Download from the web take the most of time.
Now we prevent to download data again.
Disk access is may be what take the most of time (more than parsing).
We can perhaps keep in RAM a limited number of ResponseList. For instance, if you press backspace we load and parse the file again.
I agree it can be optimized but it seems to be acceptable, even on my old laptop.
What I would work on is the timeout: when download instead of use the cached data ?</pre>
<br />
<p>- Lionel</p>
<br />
<p>On January 16th, 2011, 4:01 p.m., Lionel Chauvin wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.orgrb/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 rekonq.</div>
<div>By Lionel Chauvin.</div>
<p style="color: grey;"><i>Updated Jan. 16, 2011, 4:01 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;">it is the first step of a cache system for suggestions.
keep downloaded suggestions in memory.
prevent downloaded it again.
the completion list is more reactive.
</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>src/opensearch/opensearchengine.h <span style="color: grey">(c05501f)</span></li>
<li>src/opensearch/opensearchengine.cpp <span style="color: grey">(ef0f3f7)</span></li>
<li>src/opensearch/opensearchmanager.cpp <span style="color: grey">(41e46ec)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/100398/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>