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

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

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.