-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120250/
-----------------------------------------------------------
Review request for Calligra.
Bugs: 306000
http://bugs.kde.org/show_bug.cgi?id=306000
Repository: calligra
Description
-------
Returns an enum instead of a boolean and relying on an integer value aside.
This allows the backtrack code to know that a layout ended because of a page
break,
and thus not follow the keep with next instead of ending up in an infinite loop.
With that patch, we still have a difference between us and LibreOffice 4.3, a
check of the OpenDocument specification will perhaps help : they decide to just
skip the page break when it is in a keep with next block.
Diffs
-----
libs/textlayout/KoTextLayoutArea.h 7a15191e5a3dfb05a3e62ae7b9aaadabcceeb1fb
libs/textlayout/KoTextLayoutArea.cpp c74dbd490bf5c48386383f5d622722dfa8b5f7cc
Diff: https://git.reviewboard.kde.org/r/120250/diff/
Testing
-------
Checked with the document from bug report 306000 : layouting the document now
works and does not end up in an infinite loop.
Thanks,
Pierre Ducroquet
_______________________________________________
calligra-devel mailing list
calligra-devel@kde.org
https://mail.kde.org/mailman/listinfo/calligra-devel
