Hi, I am back from holidays. I have for the moment problems with my Virtualbox 4 installation which keeps crashing on me as soon as I push it a bit. That means I'll have difficulties for reviewing the patch right now. I am considering trying a development set-up on Windows until Virtualbox is stabilised again, but i am not sure how feasible this is (any hint in this direction is actually welcomed).<br>
<br>PierreSt<br><br><br><div class="gmail_quote">On Wed, Jan 5, 2011 at 4:44 PM, Sebastian Sauer <span dir="ltr"><<a href="mailto:mail@dipe.org">mail@dipe.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im">C. Boemann wrote:<br>
> Last week I worked on the text layout, and I'm now requesting a merge of<br>
> the branch I worked in:<br>
><br>
> text-layoutrestructure-boemann<br>
><br>
> What I've done is moving the text runaround properties from the KWFrame<br>
> class to KoShape<br>
><br>
> Secondly I moved the runaround code from KWord to the TextShape.<br>
> However it is still the responsibility of the application to supply the<br>
> textshape with the relevant shapes.<br>
><br>
> This was stepd 2-4 in my big 7 step master plan that I've talked to all<br>
> words developers about.<br>
><br>
> Please take a look, and comment.<br>
><br>
> I've made basic testing and I'm rather confident that there are no<br>
> regressions. Many unit test might be broken, and should be disabled for<br>
> now.<br>
><br>
> Review mainly requested from hanzes,pierreSt ,pinaraf, sebsauer, but also<br>
> anyone else who think they have something to contribute.<br>
<br>
</div>Review done. In words/part/tests/TestDocumentLayout.cpp before 21 tests failed<br>
and now 25 are failing. The probably most interesting one seems to be in<br>
placeAnchoredFrame3() which does;<br>
<br>
QTextLayout *lay = doc->begin().layout();<br>
QVERIFY(lay->lineCount() >= 2);<br>
<br>
Before the refactoring that lay->lineCount() call returns 17 here and after it<br>
returns 1. Also waiting for a few seconds after the layout doesn't change that<br>
result either. Can that be a problem?<br>
<br>
Beside this case it looks fine to me :-)<br>
<div><div></div><div class="h5">_______________________________________________<br>
calligra-devel mailing list<br>
<a href="mailto:calligra-devel@kde.org">calligra-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/calligra-devel" target="_blank">https://mail.kde.org/mailman/listinfo/calligra-devel</a><br>
</div></div></blockquote></div><br>