Author: spepping
Date: Mon Jan 21 11:59:49 2008
New Revision: 613989
URL: http://svn.apache.org/viewvc?rev=613989&view=rev
Log:
Undo some unnecessary changes
Modified:
xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/AbstractBreaker.java
xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/BlockKnuthSequence.java
xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/BreakingAlgorithm.java
xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/InlineKnuthSequence.java
xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/KnuthSequence.java
xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/PageBreakingAlgorithm.java
Modified:
xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/AbstractBreaker.java
URL:
http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/AbstractBreaker.java?rev=613989&r1=613988&r2=613989&view=diff
==============================================================================
---
xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/AbstractBreaker.java
(original)
+++
xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/AbstractBreaker.java
Mon Jan 21 11:59:49 2008
@@ -483,7 +483,7 @@
if (p == 0) {
lastBreakClass = effectiveList.getStartOn();
} else {
- ListElement lastBreakElement =
effectiveList.getListElement(endElementIndex);
+ ListElement lastBreakElement =
effectiveList.getElement(endElementIndex);
if (lastBreakElement.isPenalty()) {
KnuthPenalty pen = (KnuthPenalty)lastBreakElement;
lastBreakClass = pen.getBreakClass();
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=613989&r1=613988&r2=613989&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
Mon Jan 21 11:59:49 2008
@@ -163,7 +163,7 @@
public KnuthElement resolveAndGetKnuthElement(int index) {
resolveElements(index);
if (index < size()) {
- return getKnuthElement(index);
+ return (KnuthElement) get(index);
} else {
return null;
}
@@ -191,8 +191,7 @@
* @param doall resolve all elements or not
*/
private void resolveElements(int startIndex, boolean doall) {
- int i;
- for (i = startIndex; i < size(); ++i) {
+ for (int i = startIndex; i < size(); ++i) {
ListElement elt = (ListElement) get(i);
if (!doall && !elt.isUnresolvedElement()
&& !(elt instanceof LineBreakingListElement)
Modified:
xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/BreakingAlgorithm.java
URL:
http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/BreakingAlgorithm.java?rev=613989&r1=613988&r2=613989&view=diff
==============================================================================
---
xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/BreakingAlgorithm.java
(original)
+++
xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/BreakingAlgorithm.java
Mon Jan 21 11:59:49 2008
@@ -38,9 +38,6 @@
* }
* </pre>
*/
-/**
- *
- */
public abstract class BreakingAlgorithm {
/** the logger for the class */
@@ -439,7 +436,6 @@
if (alignment != org.apache.fop.fo.Constants.EN_CENTER) {
while (par.size() > firstBoxIndex) {
// scan for unresolved elements and paragraphs
- // par.resolveElements(firstBoxIndex);
thisElement = par.resolveAndGetKnuthElement(firstBoxIndex);
if (thisElement == null || thisElement.isBox()) {
break;
@@ -583,10 +579,10 @@
* @return the requested context FO note or null, if no context node could
be determined
*/
private FONode findContextFO(KnuthSequence seq, int position) {
- ListElement el = seq.getListElement(position);
+ ListElement el = seq.getElement(position);
while (el.getLayoutManager() == null && position < seq.size() - 1) {
position++;
- el = seq.getListElement(position);
+ el = seq.getElement(position);
}
Position pos = (el != null ? el.getPosition() : null);
LayoutManager lm = (pos != null ? pos.getLM() : null);
@@ -671,7 +667,7 @@
// these elements twice
int restartingIndex = restartingNode.position;
while (restartingIndex + 1 < par.size()
- && !(par.getKnuthElement(restartingIndex + 1).isBox())) {
+ && !(getElement(restartingIndex + 1).isBox())) {
restartingIndex++;
}
return restartingIndex;
@@ -817,7 +813,7 @@
// was just before the next box element, thus ignoring glues and
// penalties between the "real" break and the following box
for (int i = elementIdx; i < par.size(); i++) {
- KnuthElement tempElement = par.getKnuthElement(i);
+ KnuthElement tempElement = getElement(i);
if (tempElement.isBox()) {
break;
} else if (tempElement.isGlue()) {
@@ -949,8 +945,8 @@
}
if (element.isPenalty() && ((KnuthPenalty) element).isFlagged()
- && par.getKnuthElement(activeNode.position).isPenalty()
- && ((KnuthPenalty)
par.getKnuthElement(activeNode.position)).isFlagged()) {
+ && getElement(activeNode.position).isPenalty()
+ && ((KnuthPenalty) getElement(activeNode.position)).isFlagged()) {
// add demerit for consecutive breaks at flagged penalties
demerits += repeatedFlaggedDemerit;
// there are at least two consecutive lines ending with a flagged
penalty;
@@ -960,7 +956,7 @@
for (KnuthNode prevNode = activeNode.previous;
prevNode != null && flaggedPenaltiesCount <=
maxFlaggedPenaltiesCount;
prevNode = prevNode.previous) {
- KnuthElement prevElement =
par.getKnuthElement(prevNode.position);
+ KnuthElement prevElement = getElement(prevNode.position);
if (prevElement.isPenalty()
&& ((KnuthPenalty) prevElement).isFlagged()) {
// the previous line ends with a flagged penalty too
@@ -988,6 +984,15 @@
}
protected void finish() {
+ }
+
+ /**
+ * Return the element at index idx in the paragraph.
+ * @param idx index of the element.
+ * @return the element at index idx in the paragraph.
+ */
+ protected KnuthElement getElement(int idx) {
+ return (KnuthElement) par.get(idx);
}
/**
Modified:
xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/InlineKnuthSequence.java
URL:
http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/InlineKnuthSequence.java?rev=613989&r1=613988&r2=613989&view=diff
==============================================================================
---
xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/InlineKnuthSequence.java
(original)
+++
xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/InlineKnuthSequence.java
Mon Jan 21 11:59:49 2008
@@ -74,7 +74,7 @@
// does the first element of the first paragraph add to an existing
word?
ListElement lastOldElement, firstNewElement;
lastOldElement = getLast();
- firstNewElement = sequence.getListElement(0);
+ firstNewElement = sequence.getElement(0);
if (firstNewElement.isBox() && !((KnuthElement)
firstNewElement).isAuxiliary()
&& lastOldElement.isBox() && ((KnuthElement)
lastOldElement).getW() != 0) {
addALetterSpace();
@@ -107,9 +107,9 @@
KnuthBox prevBox = (KnuthBox) getLast();
if (prevBox.isAuxiliary()
&& (size() < 4
- || !getListElement(size() - 2).isGlue()
- || !getListElement(size() - 3).isPenalty()
- || !getListElement(size() - 4).isBox()
+ || !getElement(size() - 2).isGlue()
+ || !getElement(size() - 3).isPenalty()
+ || !getElement(size() - 4).isBox()
)
) {
// Not the sequence we are expecting
@@ -164,7 +164,7 @@
* @see org.apache.fop.layoutmgr.KnuthSequence#resolveAndGetElement(int)
*/
public KnuthElement resolveAndGetKnuthElement(int index) {
- return getKnuthElement(index);
+ return (KnuthElement) get(index);
}
/**
Modified:
xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/KnuthSequence.java
URL:
http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/KnuthSequence.java?rev=613989&r1=613988&r2=613989&view=diff
==============================================================================
---
xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/KnuthSequence.java
(original)
+++
xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/KnuthSequence.java
Mon Jan 21 11:59:49 2008
@@ -26,9 +26,6 @@
/**
* Represents a list of Knuth elements.
*/
-/**
- *
- */
public abstract class KnuthSequence extends ArrayList {
/**
* Creates a new and empty list.
@@ -166,16 +163,8 @@
* @param index The index of the element to be returned
* @return the element at index index.
*/
- public ListElement getListElement(int index) {
+ public ListElement getElement(int index) {
return (ListElement) get(index);
- }
-
- /**
- * @param index The index of the element to be returned
- * @return the element at index index.
- */
- public KnuthElement getKnuthElement(int index) {
- return (KnuthElement) get(index);
}
/**
Modified:
xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/PageBreakingAlgorithm.java
URL:
http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/PageBreakingAlgorithm.java?rev=613989&r1=613988&r2=613989&view=diff
==============================================================================
---
xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/PageBreakingAlgorithm.java
(original)
+++
xmlgraphics/fop/branches/Temp_Interleaved_Page_Line_Breaking/src/java/org/apache/fop/layoutmgr/PageBreakingAlgorithm.java
Mon Jan 21 11:59:49 2008
@@ -279,7 +279,7 @@
// remove from footnotesList the note lists that will be met
// after the restarting point
for (int j = currentIndex; j >= restartingNode.position; j--) {
- KnuthElement resettedElement = par.getKnuthElement(j);
+ KnuthElement resettedElement = getElement(j);
if (resettedElement instanceof KnuthBlockBox
&& ((KnuthBlockBox) resettedElement).hasAnchors()) {
resetFootnotes(((KnuthBlockBox)
resettedElement).getFootnoteElementLists());
@@ -407,7 +407,7 @@
int index;
// ignore suppressed elements
for (index = prevBreakIndex + 1;
- !par.getListElement(index).isBox();
+ !par.getElement(index).isBox();
index++) {
//nop
}
@@ -415,9 +415,9 @@
for (;
index < breakIndex;
index++) {
- if (par.getListElement(index).isGlue() &&
par.getListElement(index - 1).isBox()
- || par.getListElement(index).isPenalty()
- && ((KnuthElement) par.getListElement(index)).getP() <
KnuthElement.INFINITE) {
+ if (par.getElement(index).isGlue() && par.getElement(index -
1).isBox()
+ || par.getElement(index).isPenalty()
+ && ((KnuthElement) par.getElement(index)).getP() <
KnuthElement.INFINITE) {
// break found
break;
}
@@ -626,8 +626,8 @@
}
if (element.isPenalty() && ((KnuthPenalty) element).isFlagged()
- && par.getKnuthElement(activeNode.position).isPenalty()
- && ((KnuthPenalty)
par.getKnuthElement(activeNode.position)).isFlagged()) {
+ && getElement(activeNode.position).isPenalty()
+ && ((KnuthPenalty) getElement(activeNode.position)).isFlagged()) {
// add demerit for consecutive breaks at flagged penalties
demerits += repeatedFlaggedDemerit;
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
