You are viewing a plain text version of this content. The canonical link for it is here.
Posted to fop-commits@xmlgraphics.apache.org by ad...@apache.org on 2011/02/06 21:50:57 UTC
svn commit: r1067756 -
/xmlgraphics/fop/trunk/src/java/org/apache/fop/layoutmgr/SpaceResolver.java
Author: adelmelle
Date: Sun Feb 6 20:50:56 2011
New Revision: 1067756
URL: http://svn.apache.org/viewvc?rev=1067756&view=rev
Log:
Minor nits
Modified:
xmlgraphics/fop/trunk/src/java/org/apache/fop/layoutmgr/SpaceResolver.java
Modified: xmlgraphics/fop/trunk/src/java/org/apache/fop/layoutmgr/SpaceResolver.java
URL: http://svn.apache.org/viewvc/xmlgraphics/fop/trunk/src/java/org/apache/fop/layoutmgr/SpaceResolver.java?rev=1067756&r1=1067755&r2=1067756&view=diff
==============================================================================
--- xmlgraphics/fop/trunk/src/java/org/apache/fop/layoutmgr/SpaceResolver.java (original)
+++ xmlgraphics/fop/trunk/src/java/org/apache/fop/layoutmgr/SpaceResolver.java Sun Feb 6 20:50:56 2011
@@ -136,7 +136,7 @@ public final class SpaceResolver {
private String toString(Object[] arr1, Object[] arr2) {
if (arr1.length != arr2.length) {
- new IllegalArgumentException("The length of both arrays must be equal");
+ throw new IllegalArgumentException("The length of both arrays must be equal");
}
StringBuffer sb = new StringBuffer("[");
for (int i = 0; i < arr1.length; i++) {
@@ -647,7 +647,7 @@ public final class SpaceResolver {
}
}
//last = !iter.hasNext();
- if (breakPoss == null && unresolvedSecond.size() == 0 && !last) {
+ if (breakPoss == null && unresolvedSecond.isEmpty() && !last) {
LOG.trace("Swap first and second parts in no-break condition,"
+ " second part is empty.");
//The first list is reversed, so swap if this shouldn't happen
---------------------------------------------------------------------
To unsubscribe, e-mail: fop-commits-unsubscribe@xmlgraphics.apache.org
For additional commands, e-mail: fop-commits-help@xmlgraphics.apache.org