Some warnings/improvements suggested by IntelliJ

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

diff --git a/src/java/org/apache/poi/hssf/record/RecordFactory.java b/src/java/org/apache/poi/hssf/record/RecordFactory.java
index ec034d3fcb6d4d50935429bd0c206db93470286e..2336999b137681f047b817a92f1d9fc7c9b5130b 100644 (file)
--- a/src/java/org/apache/poi/hssf/record/RecordFactory.java
+++ b/src/java/org/apache/poi/hssf/record/RecordFactory.java
@@ -90,12 +90,12 @@ public final class RecordFactory {
                 throw new RuntimeException(e);
             } catch (InvocationTargetException e) {
                 Throwable t = e.getTargetException();
-                if (t instanceof RecordFormatException) {
-                    throw (RecordFormatException)t;
+                if (t instanceof org.apache.poi.util.RecordFormatException) {
+                    throw (org.apache.poi.util.RecordFormatException)t;
                 } else if (t instanceof EncryptedDocumentException) {
                     throw (EncryptedDocumentException)t;
                 } else {
-                    throw new RecordFormatException("Unable to construct record instance" , t);
+                    throw new org.apache.poi.util.RecordFormatException("Unable to construct record instance" , t);
                 }
             }
         }
@@ -123,7 +123,7 @@ public final class RecordFactory {
             } catch (IllegalAccessException e) {
                 throw new RuntimeException(e);
             } catch (InvocationTargetException e) {
-                throw new RecordFormatException("Unable to construct record instance" , e.getTargetException());
+                throw new org.apache.poi.util.RecordFormatException("Unable to construct record instance" , e.getTargetException());
             }
         }
         @Override
@@ -443,7 +443,7 @@ public final class RecordFactory {
             try {
                 sid = recClass.getField("sid").getShort(null);
             } catch (Exception illegalArgumentException) {
-                throw new RecordFormatException(
+                throw new org.apache.poi.util.RecordFormatException(
                         "Unable to determine record types");
             }
             Integer key = Integer.valueOf(sid);
@@ -482,9 +482,9 @@ public final class RecordFactory {
      *
      * @return an array of Records created from the InputStream
      *
-     * @exception RecordFormatException on error processing the InputStream
+     * @exception org.apache.poi.util.RecordFormatException on error processing the InputStream
      */
-    public static List<Record> createRecords(InputStream in) throws RecordFormatException {
+    public static List<Record> createRecords(InputStream in) throws org.apache.poi.util.RecordFormatException {
 
         List<Record> records = new ArrayList<Record>(NUM_RECORDS);
 

diff --git a/src/java/org/apache/poi/hssf/record/aggregates/ConditionalFormattingTable.java b/src/java/org/apache/poi/hssf/record/aggregates/ConditionalFormattingTable.java
index 93bae37d9d8c9b7285c3cadf68af9a2d6c540cea..cfbd36995dfbf799bee9589ae6d942cc18e18993 100644 (file)
--- a/src/java/org/apache/poi/hssf/record/aggregates/ConditionalFormattingTable.java
+++ b/src/java/org/apache/poi/hssf/record/aggregates/ConditionalFormattingTable.java
@@ -51,8 +51,7 @@ public final class ConditionalFormattingTable extends RecordAggregate {
        }
 
        public void visitContainedRecords(RecordVisitor rv) {
-               for (int i = 0; i < _cfHeaders.size(); i++) {
-                       CFRecordsAggregate subAgg = _cfHeaders.get(i);
+               for (CFRecordsAggregate subAgg : _cfHeaders) {
                        subAgg.visitContainedRecords(rv);
                }
        }

diff --git a/src/testcases/org/apache/poi/ss/format/TestCellFormatCondition.java b/src/testcases/org/apache/poi/ss/format/TestCellFormatCondition.java
index f1ede1193c3ab90b2f7c72e18d346af63aada929..1a56b824a93e3fed89f3fb40da9ed9ad0121d126 100644 (file)
--- a/src/testcases/org/apache/poi/ss/format/TestCellFormatCondition.java
+++ b/src/testcases/org/apache/poi/ss/format/TestCellFormatCondition.java
@@ -17,7 +17,6 @@
 package org.apache.poi.ss.format;
 
 import junit.framework.TestCase;
-import org.apache.poi.ss.format.CellFormatCondition;
 
 public class TestCellFormatCondition extends TestCase {
     public void testSVConditions() {