Review Request 120250: Change blockLayout return to be more explicit

[Pierre Ducroquet]

-----------------------------------------------------------
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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20140917/bd46f007/attachment.htm>