<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/102093/">http://git.reviewboard.kde.org/r/102093/</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>
<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.
Compared to the previous review-request ( https://git.reviewboard.kde.org/r/102028/diff/ ) we now also handle the case that Qt doesn't support threaded font rendering what could be a problem cause some of the VectorShape-backends are using QPainter::drawText. As alternate I tried to use QPainterPath as suggested at http://www.qtcentre.org/archive/index.php/t-6542.html but the results are rather bad. Looks like text rendered with QPainterPath has "quantisationeffects" (not sure if that's a proper translation to english) even if AntiAliasing is enabled (testapp is at http://paste.kde.org/101725/ ).
</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/102093/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>