FOP-2179: Fix checkstyle and findbugs warnings.

git-svn-id: https://svn.apache.org/repos/asf/xmlgraphics/fop/trunk@1427143 13f79535-47bb-0310-9956-ffa450edef68

5 files changed, 14 insertions, 10 deletions

diff --git a/src/java/org/apache/fop/area/inline/InlineArea.java b/src/java/org/apache/fop/area/inline/InlineArea.java
index d92ea4f4f..399bd1f1e 100644
--- a/src/java/org/apache/fop/area/inline/InlineArea.java
+++ b/src/java/org/apache/fop/area/inline/InlineArea.java
@@ -278,7 +278,7 @@ public class InlineArea extends Area {
             storedIPDVariation += ipdVariation;
         }
     }
-    
+
     /**
      * Returns the offset that this area would have if its offset and size were taking
      * children areas into account. The bpd of an inline area is taken from its nominal
diff --git a/src/java/org/apache/fop/area/inline/InlineParent.java b/src/java/org/apache/fop/area/inline/InlineParent.java
index 4d3f27b30..1be3aff0f 100644
--- a/src/java/org/apache/fop/area/inline/InlineParent.java
+++ b/src/java/org/apache/fop/area/inline/InlineParent.java
@@ -19,10 +19,9 @@
 
 package org.apache.fop.area.inline;
 
-import java.util.Iterator;
 import java.util.ArrayList;
+import java.util.Iterator;
 import java.util.List;
-import java.lang.Math;
 
 import org.apache.fop.area.Area;
 
@@ -41,10 +40,10 @@ public class InlineParent extends InlineArea {
 
     /** Controls whether the IPD is automatically adjusted based on the area's children. */
     protected transient boolean autoSize;
-    
+
     /** The offset of the <q>beforest</q> child area of this area. */
     protected int minChildOffset;
-    
+
     /**
      * The offset of the <q>afterest</q> child area of this area. Offset from the
      * before-edge of this area's content-rectangle and the after-edge of the child area's
diff --git a/src/java/org/apache/fop/fo/FObj.java b/src/java/org/apache/fop/fo/FObj.java
index d50e18b0e..b19ccbe20 100644
--- a/src/java/org/apache/fop/fo/FObj.java
+++ b/src/java/org/apache/fop/fo/FObj.java
@@ -25,6 +25,7 @@ import java.util.Iterator;
 import java.util.List;
 import java.util.ListIterator;
 import java.util.Map;
+import java.util.Map.Entry;
 import java.util.NoSuchElementException;
 import java.util.Set;
 
@@ -132,12 +133,13 @@ public abstract class FObj extends FONode implements Constants {
     private void warnOnUnknownProperties(Attributes attlist, String objName, PropertyList propertyList)
             throws FOPException {
         Map<String, Property> unknowns = propertyList.getUnknownPropertyValues();
-        for (String propertyValue : unknowns.keySet()) {
+        for (Entry<String, Property> value : unknowns.entrySet()) {
+            String unknownProperty = value.getKey();
             FOValidationEventProducer producer = FOValidationEventProducer.Provider.get(getUserAgent()
                     .getEventBroadcaster());
             producer.invalidPropertyValue(this, objName,
-                    getAttributeNameForValue(attlist, unknowns.get(propertyValue), propertyList),
-                    propertyValue, null,
+                    getAttributeNameForValue(attlist, unknowns.get(unknownProperty), propertyList),
+                    unknownProperty, null,
                     getLocator());
         }
     }
diff --git a/src/java/org/apache/fop/layoutmgr/BalancingColumnBreakingAlgorithm.java b/src/java/org/apache/fop/layoutmgr/BalancingColumnBreakingAlgorithm.java
index f40a44d0e..06c3c538d 100644
--- a/src/java/org/apache/fop/layoutmgr/BalancingColumnBreakingAlgorithm.java
+++ b/src/java/org/apache/fop/layoutmgr/BalancingColumnBreakingAlgorithm.java
@@ -63,8 +63,8 @@ public class BalancingColumnBreakingAlgorithm extends PageBreakingAlgorithm {
             return demerits;
         }
         for (int breakPos = 0; breakPos < curPossibility.size(); breakPos++) {
-            if (curPossibility.get(breakPos) != 0 &&
-                    !curPossibility.get(breakPos).equals(idealBreaks.get(breakPos))) {
+            if (curPossibility.get(breakPos) != 0
+                    && !curPossibility.get(breakPos).equals(idealBreaks.get(breakPos))) {
                 notIdeal = true;
                 break;
             }
diff --git a/status.xml b/status.xml
index cd7feb080..394c0bf89 100644
--- a/status.xml
+++ b/status.xml
@@ -59,6 +59,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="FOP-2179" due-to="Robert Meyer">
+        Fix checkstyle and findbugs warnings.
+      </action>
       <action context="Layout" dev="GA" type="fix" fixes-bug="FOP-2160" due-to="Robert Meyer">
         Fix NPE when rl writing mode is used in certain contexts.
       </action>