<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://reviewboard.kde.org/r/4645/">http://reviewboard.kde.org/r/4645/</a>
</td>
</tr>
</table>
<br />
<p>Ship it!</p>
<pre>the code is more readable imho after your changes. nice :)
what, btw, were the small bugs that were fixed?
i did notice that the clipping isn't always turned on in the painter; that was previously avoided if possible since clipping adds a bunch more operations on the painting. whether or not this creates a meaningful difference is something that ought to be measured. before committing, please put a TODO note above the clip rect line noting that the impact of always setting the clip rect should be measured to ensure that it is negligible enough not to worry about.</pre>
<br />
<p>- Aaron</p>
<br />
<p>On July 15th, 2010, 2:39 p.m., Manuel Mommertz wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://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 Manuel Mommertz.</div>
<p style="color: grey;"><i>Updated 2010-07-15 14:39:18</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;">This patch mainly moves lines around so it is easier to read (in my opinion). While doing this I found two little bugs that are solved in this. Of cause if you don't like this relayout I will fix the bugs separately.</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>/trunk/KDE/kdebase/workspace/plasma/generic/applets/systemtray/ui/applet.cpp <span style="color: grey">(1150203)</span></li>
</ul>
<p><a href="http://reviewboard.kde.org/r/4645/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>