diff options
| author | Glenn Adams <gadams@apache.org> | 2012-04-21 04:45:14 +0000 |
|---|---|---|
| committer | Glenn Adams <gadams@apache.org> | 2012-04-21 04:45:14 +0000 |
| commit | 0b18292f887bb18afb46eb75ae92d7731ee5644b (patch) | |
| tree | 0961686b924ebcb06b91a83d558c315e478c337e | |
| parent | f5141241f7f7597c16bd3b030fa755a9dcec43d9 (diff) | |
| download | xmlgraphics-fop-0b18292f887bb18afb46eb75ae92d7731ee5644b.tar.gz xmlgraphics-fop-0b18292f887bb18afb46eb75ae92d7731ee5644b.zip | |
Bugzilla #51043: Don't restart layout unless abs(ipd difference)>1 in order to prevent rounding issues from triggering false restart.
git-svn-id: https://svn.apache.org/repos/asf/xmlgraphics/fop/trunk@1328581 13f79535-47bb-0310-9956-ffa450edef68
| -rw-r--r-- | src/java/org/apache/fop/layoutmgr/AbstractBreaker.java | 2 | ||||
| -rw-r--r-- | status.xml | 3 |
2 files changed, 4 insertions, 1 deletions
diff --git a/src/java/org/apache/fop/layoutmgr/AbstractBreaker.java b/src/java/org/apache/fop/layoutmgr/AbstractBreaker.java index dc87f9f1d..5fa411101 100644 --- a/src/java/org/apache/fop/layoutmgr/AbstractBreaker.java +++ b/src/java/org/apache/fop/layoutmgr/AbstractBreaker.java @@ -420,7 +420,7 @@ public abstract class AbstractBreaker { alg.setConstantLineWidth(flowBPD); int optimalPageCount = alg.findBreakingPoints(effectiveList, 1, true, BreakingAlgorithm.ALL_BREAKS); - if (alg.getIPDdifference() != 0) { + if ( Math.abs ( alg.getIPDdifference() ) > 1 ) { addAreas(alg, optimalPageCount, blockList, effectiveList); // *** redo Phase 1 *** log.trace("IPD changes after page " + optimalPageCount); diff --git a/status.xml b/status.xml index 84708802e..ff2782c38 100644 --- a/status.xml +++ b/status.xml @@ -62,6 +62,9 @@ documents. Example: the fix of marks layering will be such a case when it's done. --> <release version="FOP Trunk" date="TBD"> + <action context="Code" dev="GA" type="fix" fixes-bug="51043" due-to="Pascal Sancho"> + Don't restart layout unless abs(ipd difference) > 1 in order to prevent rounding issues from triggering false restart. + </action> <action context="Fonts" dev="GA" type="update"> Removing experimental feature that violates XSL-FO and Unicode semantics by misinterpreting Basic Latin code points. Users must use private use codepoints to access font specific character mappings that have no assigned Unicode code point. See bug 50492. |