<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/101111/">http://git.reviewboard.kde.org/r/101111/</a>
</td>
</tr>
</table>
<br />
<p>Ship it!</p>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I have tested again and the deferred loading is working now. Please commit.</pre>
<br />
<p>- Thorsten</p>
<br />
<p>On April 13th, 2011, 1:02 p.m., Boudewijn Rempt 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 Boudewijn Rempt.</div>
<p style="color: grey;"><i>Updated April 13, 2011, 1:02 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;">This patch implements three things:
* deferred plugin loading for shape plugins. Shapes (even with tools) can now be loaded in two stages: a stub factory which defers to the second, real plugin. The advantage is that we do not load all libraries of all Calligra applications when starting a calligra application. The disadvantage is slightly more complexity. For unported plugins, nothing changes at all.
* dynamic tool insertion. Until now, tool factories were loaded at at startup and tools were created on canvas creation. Now it is possible to dynamically add a tool factory. Doing so will create a new tool for all canvases.
* split the spreadsheet shape plugin as an example.
The text shape plugin is next, in this patch it's still disabled.</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/flake/KoDeferredShapeFactoryBase.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>libs/flake/CMakeLists.txt <span style="color: grey">(e4a16f3a34b5e4645a0bb60898926ccc0f7bfb0d)</span></li>
<li>krita/plugins/assistants/RulerAssistant/SplineAssistant.cc <span style="color: grey">(7c212d94379a73d68255109e55e7641f4d3a33de)</span></li>
<li>krita/image/tiles3/kis_tile_data.h <span style="color: grey">(d00a4afc11952c3df16abe5ff89f85be92dfa4d5)</span></li>
<li>krita/plugins/assistants/RulerAssistant/PerspectiveAssistant.cc <span style="color: grey">(f49842c90fc712165520e588fd26c71126795ec5)</span></li>
<li>libs/flake/KoDeferredShapeFactoryBase.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>libs/flake/KoShapeFactoryBase.h <span style="color: grey">(8ff17e4d27a4c87db280bdbfa484c81dafffe66d)</span></li>
<li>libs/flake/KoShapeFactoryBase.cpp <span style="color: grey">(536b0baef75cc3de5073ae90cbcb9eedeabae0fe)</span></li>
<li>libs/flake/KoToolManager.h <span style="color: grey">(00333248644cf4a1021d55623c5061a580229ad6)</span></li>
<li>libs/flake/KoToolManager.cpp <span style="color: grey">(3b6554a41dc4910b238473477b59f02b7c0c52de)</span></li>
<li>libs/flake/KoToolRegistry.h <span style="color: grey">(bcdb5de89e089e850e7d93eb6fd95a42e5b3b110)</span></li>
<li>libs/flake/KoToolRegistry.cpp <span style="color: grey">(f83072db104144035c350b6056bdc02081e1fb25)</span></li>
<li>libs/main/CMakeLists.txt <span style="color: grey">(a7f3077db8db5be3cf9a77f77e87c09fc84f17a0)</span></li>
<li>libs/main/KoToolBox.cpp <span style="color: grey">(092920b3f1b5a25f63df81125df2d3565bb63b46)</span></li>
<li>libs/main/KoToolBoxDocker.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>libs/main/KoToolBoxDocker_p.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>libs/main/KoToolBoxFactory.h <span style="color: grey">(8269759d5ccdf728007a6e613ce18feeda527c20)</span></li>
<li>libs/main/KoToolBoxFactory.cpp <span style="color: grey">(4af151dcdfe373ae2697f65940b223d811ea8f53)</span></li>
<li>libs/main/KoToolBoxLayout_p.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>libs/main/KoToolBox_p.h <span style="color: grey">(81e8178a8f62b40f5a6bb4201b45e34bd4a95021)</span></li>
<li>libs/main/KoToolDocker_p.h <span style="color: grey">(3f965dd29523b73fcf1f7b2133366c8e514db627)</span></li>
<li>plugins/CMakeLists.txt <span style="color: grey">(6f3b538850909c6bb34be7845531582f46df125c)</span></li>
<li>plugins/artistictextshape/MoveStartOffsetStrategy.cpp <span style="color: grey">(6a47031ddd0dd33b89897cfcde6df67bf4865787)</span></li>
<li>plugins/textshape/CMakeLists.txt <span style="color: grey">(3eb805bee8e4b81faae4b767380b65770d4ce52a)</span></li>
<li>plugins/textshape/TextShapeDeferredFactory.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/textshape/TextShapeDeferredFactory.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>plugins/textshape/TextShapeFactory.h <span style="color: grey">(91202a01534cfe93fff4abaf573ec32b009335c2)</span></li>
<li>plugins/textshape/TextShapeFactory.cpp <span style="color: grey">(d65c2c87f1441c672a0f3378d6e170d79cd05ae7)</span></li>
<li>plugins/textshape/textshape-deferred.desktop <span style="color: grey">(PRE-CREATION)</span></li>
<li>servicetypes/CMakeLists.txt <span style="color: grey">(3db70a8fc61b638b33cf4c7bca3cc3b5d30a88cd)</span></li>
<li>servicetypes/README.txt <span style="color: grey">(PRE-CREATION)</span></li>
<li>servicetypes/calligra_deferred_plugin.desktop <span style="color: grey">(PRE-CREATION)</span></li>
<li>tables/shape/CMakeLists.txt <span style="color: grey">(07a2109a2b1b1d50b06c51c94b6f7c39162419f5)</span></li>
<li>tables/shape/TableShapeDeferredFactory.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>tables/shape/TableShapeDeferredFactory.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>tables/shape/TableShapeFactory.h <span style="color: grey">(effadbae553233c55cf1c2bd1142973e87010504)</span></li>
<li>tables/shape/TableShapeFactory.cpp <span style="color: grey">(a62c48acac2c1ef8c027e370107bcb58bf9c8fbf)</span></li>
<li>tables/shape/spreadsheetshape-deferred.desktop <span style="color: grey">(PRE-CREATION)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/101111/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>