<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/100438/">http://git.reviewboard.kde.org/r/100438/</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;">Okay, I looked at it good and hard. The patch is actually ok, and doesn't harm the gui update throttling (I had forgotten I had implemented that in a different way). However, the call to addPoint() should definitely be made option, for instance by adding a parameter to KoProgressUpdater's constructor that enables profiling, or an environment settings.
It's useless to store all these timepoints when it isn't needed at all, and takes a lot of time, as confirmed by Marijn.</pre>
<br />
<p>- Boudewijn</p>
<br />
<p>On January 24th, 2011, 4:33 a.m., Thorsten Zachmann 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 Calligra.</div>
<div>By Thorsten Zachmann.</div>
<p style="color: grey;"><i>Updated Jan. 24, 2011, 4:33 a.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;">Send progress updates more frequently. There is already a limitation on what is send to the user in KoProgressUpdater that only sends updates only every 0.1 second. Without the change it can happen that the progress is only reported every 0.4 seconds. With this change the progress bar advances more smooth and does not jump as without the change</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>libs/main/KoUpdater.cpp <span style="color: grey">(8ea5fce)</span></li>
<li>libs/main/KoUpdaterPrivate_p.h <span style="color: grey">(af82419)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/100438/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>