<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/102562/">http://git.reviewboard.kde.org/r/102562/</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 b709a77144bbbf30cfc68c94bca56f9a2c15a446 by Thorsten Zachmann to branch master.</pre>
<br />
<p>- Commit</p>
<br />
<p>On September 11th, 2011, 4:49 a.m., Thorsten Zachmann 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 Thorsten Zachmann.</div>
<p style="color: grey;"><i>Updated Sept. 11, 2011, 4:49 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;">This patch makes it possible to get all gradients, markers, ... from the styles reader. It is needed when we don't want to load the same stuff over and over again.
I will use it in my marker branch to get all markers and load them in one go and just reuse them later.
There is no change in functionality for the extiting applications.</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;">Run cstester and no problems found in the documents of my test set (~1500 docs)</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/odf/KoOdfStylesReader.h <span style="color: grey">(acab80f)</span></li>
<li>libs/odf/KoOdfStylesReader.cpp <span style="color: grey">(5a8324e)</span></li>
<li>plugins/chartshape/Surface.cpp <span style="color: grey">(e22a6b2)</span></li>
<li>libs/flake/KoPatternBackground.cpp <span style="color: grey">(4210ab48)</span></li>
<li>libs/odf/KoOdfGraphicStyles.cpp <span style="color: grey">(9e9653f)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/102562/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>