IntelliJ warnings, deprecation

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

diff --git a/src/java/org/apache/poi/hssf/usermodel/HSSFCell.java b/src/java/org/apache/poi/hssf/usermodel/HSSFCell.java
index 23ed7dc7bc9671f6e94ba4e24741ce9e4aa2428d..ccfddf4a7682c7a696dd07c614e7aa66e4f8181d 100644 (file)
--- a/src/java/org/apache/poi/hssf/usermodel/HSSFCell.java
+++ b/src/java/org/apache/poi/hssf/usermodel/HSSFCell.java
@@ -53,7 +53,6 @@ import org.apache.poi.ss.util.CellAddress;
 import org.apache.poi.ss.util.CellRangeAddress;
 import org.apache.poi.ss.util.CellReference;
 import org.apache.poi.ss.util.NumberToTextConverter;
-import org.apache.poi.util.Internal;
 import org.apache.poi.util.LocaleUtil;
 import org.apache.poi.util.Removal;
 
@@ -306,7 +305,7 @@ public class HSSFCell implements Cell {
                 break;
 
             case NUMERIC :
-                NumberRecord nrec = null;
+                NumberRecord nrec;
 
                 if (cellType != _cellType)
                 {
@@ -356,7 +355,7 @@ public class HSSFCell implements Cell {
                 break;
 
             case BLANK :
-                BlankRecord brec = null;
+                BlankRecord brec;
 
                 if (cellType != _cellType)
                 {
@@ -375,7 +374,7 @@ public class HSSFCell implements Cell {
                 break;
 
             case BOOLEAN :
-                BoolErrRecord boolRec = null;
+                BoolErrRecord boolRec;
 
                 if (cellType != _cellType)
                 {
@@ -396,7 +395,7 @@ public class HSSFCell implements Cell {
                 break;
 
             case ERROR :
-                BoolErrRecord errRec = null;
+                BoolErrRecord errRec;
 
                 if (cellType != _cellType)
                 {
@@ -574,7 +573,7 @@ public class HSSFCell implements Cell {
         if (_cellType != CellType.STRING) {
             setCellType(CellType.STRING, false, row, col, styleIndex);
         }
-        int index = 0;
+        int index;
 
         HSSFRichTextString hvalue = (HSSFRichTextString) value;
         UnicodeString str = hvalue.getUnicodeString();
@@ -1132,8 +1131,6 @@ public class HSSFCell implements Cell {
      * on the cached value of the formula
      *
      * @since POI 4.0
-     * @return <code>CellType</code> depending
-     * on the cached value of the formula
      */
     @Override
     public CellType getCachedFormulaResultType() {
@@ -1182,10 +1179,7 @@ public class HSSFCell implements Cell {
     }
 
     public boolean isPartOfArrayFormulaGroup() {
-        if (_cellType != CellType.FORMULA) {
-            return false;
-        }
-        return ((FormulaRecordAggregate)_record).isPartOfArrayFormula();
+        return _cellType == CellType.FORMULA && ((FormulaRecordAggregate) _record).isPartOfArrayFormula();
     }
 
     /**

diff --git a/src/java/org/apache/poi/hssf/usermodel/HSSFName.java b/src/java/org/apache/poi/hssf/usermodel/HSSFName.java
index bbd48c23ff1e5d990480cd6ff8fa7266168dd211..d2f8b857f48f26e134775470407917151a2014f5 100644 (file)
--- a/src/java/org/apache/poi/hssf/usermodel/HSSFName.java
+++ b/src/java/org/apache/poi/hssf/usermodel/HSSFName.java
@@ -175,8 +175,6 @@ public final class HSSFName implements Name {
      * Case sensitivity: all names are case-insensitive
      * 
      * Uniqueness: must be unique (for names with the same scope)
-     *
-     * @param name
      */
     private static void validateName(String name) {
 
@@ -267,11 +265,9 @@ public final class HSSFName implements Name {
     }
 
     public String toString() {
-        StringBuffer sb = new StringBuffer(64);
-        sb.append(getClass().getName()).append(" [");
-        sb.append(_definedNameRec.getNameText());
-        sb.append("]");
-        return sb.toString();
+        return getClass().getName() + " [" +
+                _definedNameRec.getNameText() +
+                "]";
     }
 
     /**

diff --git a/src/java/org/apache/poi/hssf/util/CellReference.java b/src/java/org/apache/poi/hssf/util/CellReference.java
index ac3e5f388a2e1217563443b4705e3ea94598bbbb..3360d5f97810c4d00d89f00a4a3e36421245fe99 100644 (file)
--- a/src/java/org/apache/poi/hssf/util/CellReference.java
+++ b/src/java/org/apache/poi/hssf/util/CellReference.java
@@ -21,6 +21,10 @@ package org.apache.poi.hssf.util;
  * Common conversion functions between Excel style A1, C27 style
  *  cell references, and POI usermodel style row=0, column=0
  *  style references.
+ *
+ *  @deprecated POI 4.0.0-SNAPSHOT. Use {@link org.apache.poi.ss.util.CellReference} instead.
+ *          Actually the replacement CellReference was added earlier, but the deprecation notice
+ *          was added only in 4.0.0-SNAPSHOT.
  */
 public final class CellReference extends org.apache.poi.ss.util.CellReference {
     /**

diff --git a/src/java/org/apache/poi/ss/usermodel/CellValue.java b/src/java/org/apache/poi/ss/usermodel/CellValue.java
index a5ad66563dc327a0894d915189cd13b82c6db911..516b0ade9df39492fd40cbf997c42412a7b0142d 100644 (file)
--- a/src/java/org/apache/poi/ss/usermodel/CellValue.java
+++ b/src/java/org/apache/poi/ss/usermodel/CellValue.java
@@ -111,11 +111,9 @@ public final class CellValue {
        }
 
        public String toString() {
-               StringBuffer sb = new StringBuffer(64);
-               sb.append(getClass().getName()).append(" [");
-               sb.append(formatAsString());
-               sb.append("]");
-               return sb.toString();
+               return getClass().getName() + " [" +
+                               formatAsString() +
+                               "]";
        }
 
        public String formatAsString() {

diff --git a/src/java/org/apache/poi/ss/usermodel/FormulaEvaluator.java b/src/java/org/apache/poi/ss/usermodel/FormulaEvaluator.java
index b60e55980469c0c9e268971a835e1ac7bdf05311..860755d15d0efc1e37903ba61edb3b65c4b9af9f 100644 (file)
--- a/src/java/org/apache/poi/ss/usermodel/FormulaEvaluator.java
+++ b/src/java/org/apache/poi/ss/usermodel/FormulaEvaluator.java
@@ -76,7 +76,7 @@ public interface FormulaEvaluator {
      * the cell and also its cell type. This method should be preferred over
      * evaluateInCell() when the call should not modify the contents of the
      * original cell.
-     * @param cell
+     * @param cell The {@link Cell} to evaluate
      */
     CellValue evaluate(Cell cell);
 
@@ -144,7 +144,7 @@ public interface FormulaEvaluator {
      * Be aware that your cell value will be changed to hold the
      *  result of the formula. If you simply want the formula
      *  value computed for you, use {@link #evaluateFormulaCell(Cell)}
-     * @param cell
+     * @param cell The {@link Cell} to evaluate and modify.
      */
     Cell evaluateInCell(Cell cell);