xmlgraphics-fop-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From spepp...@apache.org
Subject svn commit: r627828 - /xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/BlockKnuthSequence.java
Date Thu, 14 Feb 2008 18:32:57 GMT
Author: spepping
Date: Thu Feb 14 10:32:55 2008
New Revision: 627828

URL: http://svn.apache.org/viewvc?rev=627828&view=rev
Log:
Add a warning

Modified:
    xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/BlockKnuthSequence.java

Modified: xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/BlockKnuthSequence.java
URL: http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/BlockKnuthSequence.java?rev=627828&r1=627827&r2=627828&view=diff
==============================================================================
--- xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/BlockKnuthSequence.java
(original)
+++ xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/BlockKnuthSequence.java
Thu Feb 14 10:32:55 2008
@@ -119,6 +119,8 @@
         Position firstPos = firstElt.getPosition();
         LayoutManager firstLM = firstPos.getLM();
         if (firstPos instanceof NonLeafPosition) {
+            // TODO: this branch is not really tested; should the subposition be null?
+            // or do elements of a paragraph always have leaf positions?
             Position firstSubPos = firstPos.getPosition();
             newPos = new NonLeafPosition(firstLM, firstSubPos);
         } else {
@@ -153,6 +155,8 @@
         Position lastPos = lastElt.getPosition();
         LayoutManager lastLM = lastPos.getLM();
         if (lastPos instanceof NonLeafPosition) {
+            // TODO: this branch is not really tested; should the subposition be null?
+            // or do elements of a paragraph always have leaf positions?
             Position lastSubPos = lastPos.getPosition();
             newPos = new NonLeafPosition(lastLM, lastSubPos);
         } else {



---------------------------------------------------------------------
To unsubscribe, e-mail: fop-commits-unsubscribe@xmlgraphics.apache.org
For additional commands, e-mail: fop-commits-help@xmlgraphics.apache.org


Mime
View raw message