<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/102028/">http://git.reviewboard.kde.org/r/102028/</a>
</td>
</tr>
</table>
<br />
<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 Sebastian Sauer.</div>
<p style="color: grey;"><i>Updated July 21, 2011, 9:05 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;">Attachd patch makes the probably rather expensive initial parsing and drawing on the QImage operations in the Vectorshape threaded.
The thread will only be used to produce the QImage we are using later on to display the vectorshape. Drawing of the QImage itself is not threaded but then that's not an expensive operation like parsing and producing those QImage can be.
UPDATE:
Seems there is the special case of drawText used in OutputPainterStrategy::extTextOut and SvmPainterBackend::textArray that is not threadsafe under X11 according to http://doc.trolltech.com/4.7/threads-modules.html#painting-in-threads and http://www.qtcentre.org/archive/index.php/t-6542.html . So, the patch next to be extended to use the QPainterPath solution named in the second link. Discarding the review-request therefore.
</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>plugins/vectorshape/VectorShape.h <span style="color: grey">(4229842)</span></li>
<li>plugins/vectorshape/VectorShape.cpp <span style="color: grey">(0a28dac)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/102028/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>