<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/111048/">http://git.reviewboard.kde.org/r/111048/</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;">This review has been submitted with commit 0af45dd5bf684d096662331904697035a5621a73 by Dawit Alemayehu to branch KDE/4.10.</pre>
<br />
<p>- Commit</p>
<br />
<p>On June 18th, 2013, 1:15 p.m. UTC, Dawit Alemayehu wrote:</p>
<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 KDE Base Apps and David Faure.</div>
<div>By Dawit Alemayehu.</div>
<p style="color: grey;"><i>Updated June 18, 2013, 1:15 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;">The attached patch changes KonqSessionManager such that it creates an instance of the auto save session KConfig object once during initialization instead of each time it is invoked, which is about every 10 secs. This stops the flood of "stat" calls from Konqueror that occur every 10 secs even when Konqueror is doing absolutely nothing. The problem and its fix can be tested by running Konqueror through strace, "strace -e stat konqueror".
The result of this fix is the number of stat calls Konqueror makes every 10 secs went down to 1 from 15. Even that remaining stat call can be stopped if I was certain that removing the "deleteOwnedSessions" call from autoSaveSession won't cause any regression. David?</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>konqueror/src/konqsessionmanager.h <span style="color: grey">(930a2c3)</span></li>
<li>konqueror/src/konqsessionmanager.cpp <span style="color: grey">(1b99cb7)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/111048/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>