<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/101803/">http://git.reviewboard.kde.org/r/101803/</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 66c0cb7887445ee99245d53ba15a6fb1cc9ec17e by Jaime Torres to branch master.</pre>
<br />
<p>- Commit</p>
<br />
<p>On June 29th, 2011, 8:32 p.m., Jaime Torres Amate wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/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 Jaime Torres Amate.</div>
<p style="color: grey;"><i>Updated June 29, 2011, 8:32 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 changes are:
(easy)
1. in formulas is to return the FormulaToken& as recommended by C++. (even if there is no a=b=c chain).
2. in ustring is to compare with NaN to know when a value is NaN instead of compare to itself.
3. in kis_blur_effect is to clarify the priority of - and ?.
(hard)
4. in kotextloader to remove 3 duplicated code. (at least the first and sencond, the third one is sligly different from the previous incarnation of the code).
They are previously in the same else if chain.
5. in kotablerowstyle, remove 1 duplicated code (or is it missing some option?)
6. in kodfgraphicstyles, remove 1 dead entry (or it should compare to some other word?)
</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>filters/tables/excel/sidewinder/formulas.h <span style="color: grey">(98559c8)</span></li>
<li>filters/tables/excel/sidewinder/formulas.cpp <span style="color: grey">(5903dd6)</span></li>
<li>filters/words/msword-odf/wv2/src/ustring.cpp <span style="color: grey">(d9f1707)</span></li>
<li>krita/plugins/filters/blur/kis_blur_filter.cpp <span style="color: grey">(3a45bc2)</span></li>
<li>libs/kotext/opendocument/KoTextLoader.cpp <span style="color: grey">(ce7c7a3)</span></li>
<li>libs/kotext/styles/KoTableRowStyle.cpp <span style="color: grey">(6fc53c6)</span></li>
<li>libs/odf/KoOdfGraphicStyles.cpp <span style="color: grey">(f5860b9)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/101803/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>