<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://svn.reviewboard.kde.org/r/5416/">http://svn.reviewboard.kde.org/r/5416/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://svn.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 Plasma.</div>
<div>By Aaron Seigo.</div>
<p style="color: grey;"><i>Updated 2010-09-23 06:39:30.784225</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;">This patch implements shared FrameData between FrameSvg's. It uses a per-FrameSvg count since it is feasible that one FrameSvg could reference the same FrameData more than once. The results are reasonable: ~25% fewer pixmaps in a default plasma-desktop startup, %57 fewer pixmaps with a layout of 4 buttons, 4 line edits and a combobox in a grid layout. This results in fewer xserver roundtrips on x11 and less memory usage on all platforms.
There is a #define near the dtor of FrameSvgPrivate that can be uncommented to see the benefits and confirm that all shared pages are indeed removed at the end. Confirming with ktracepixmap was also done.</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;">Full plasma-desktop session, test plasmoid with the buttons'n'labels torture test (over 100 of each in a grid layout).</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> (updated)</h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>/trunk/KDE/kdelibs/plasma/framesvg.h <span style="color: grey">(1178444)</span></li>
<li>/trunk/KDE/kdelibs/plasma/framesvg.cpp <span style="color: grey">(1178444)</span></li>
<li>/trunk/KDE/kdelibs/plasma/private/framesvg_p.h <span style="color: grey">(1178444)</span></li>
</ul>
<p><a href="http://svn.reviewboard.kde.org/r/5416/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>