]> source.dussan.org Git - poi.git/commitdiff
IntelliJ warnings, Javadoc, Formatting
authorDominik Stadler <centic@apache.org>
Thu, 26 Jan 2017 19:58:46 +0000 (19:58 +0000)
committerDominik Stadler <centic@apache.org>
Thu, 26 Jan 2017 19:58:46 +0000 (19:58 +0000)
Changes for the RecordFormatException deprecation

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

15 files changed:
src/java/org/apache/poi/hpsf/VariantSupport.java
src/java/org/apache/poi/hssf/record/aggregates/PageSettingsBlock.java
src/java/org/apache/poi/hssf/usermodel/HSSFCellStyle.java
src/java/org/apache/poi/hssf/util/HSSFColor.java
src/java/org/apache/poi/util/HexRead.java
src/ooxml/java/org/apache/poi/xdgf/util/VsdxToPng.java
src/ooxml/java/org/apache/poi/xssf/streaming/SXSSFWorkbook.java
src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFWorkbook.java
src/scratchpad/src/org/apache/poi/hwpf/converter/AbstractWordConverter.java
src/scratchpad/src/org/apache/poi/hwpf/converter/WordToHtmlConverter.java
src/scratchpad/src/org/apache/poi/hwpf/dev/HWPFLister.java
src/scratchpad/src/org/apache/poi/hwpf/extractor/WordExtractor.java
src/scratchpad/src/org/apache/poi/hwpf/usermodel/Picture.java
src/scratchpad/testcases/org/apache/poi/hwpf/converter/TestWordToHtmlConverter.java
src/testcases/org/apache/poi/hssf/record/aggregates/TestPageSettingsBlock.java

index 25c44dff92569102f744cf60ec3a7524d3f42eee..1181437bcf2437cfb0a77969d2b4f48519a474fe 100644 (file)
@@ -56,7 +56,7 @@ public class VariantSupport extends Variant {
 
     
     private static final POILogger logger = POILogFactory.getLogger(VariantSupport.class);
-    private static boolean logUnsupportedTypes = false;
+    private static boolean logUnsupportedTypes;
 
     /**
      * Keeps a list of the variant types an "unsupported" message has already
@@ -196,7 +196,7 @@ public class VariantSupport extends Variant {
                 return uniString.toJavaString();
 
             // if(l1 < 0) {
-            /**
+            /*
              * YK: reading the ClipboardData packet (VT_CF) is not quite
              * correct. The size of the data is determined by the first four
              * bytes of the packet while the current implementation calculates
index b42f09b47f081c0006601c494692ef7f4f58e00c..e1d21cf97d68b7463b6bca588c11fa321704220e 100644 (file)
@@ -222,7 +222,7 @@ public final class PageSettingsBlock extends RecordAggregate {
 
     private void checkNotPresent(Record rec) {
         if (rec != null) {
-            throw new RecordFormatException("Duplicate PageSettingsBlock record (sid=0x"
+            throw new org.apache.poi.util.RecordFormatException("Duplicate PageSettingsBlock record (sid=0x"
                     + Integer.toHexString(rec.getSid()) + ")");
         }
     }
@@ -614,7 +614,7 @@ public final class PageSettingsBlock extends RecordAggregate {
             throw new IllegalStateException("This page settings block already has a header/footer record");
         }
         if (rec.getSid() != HeaderFooterRecord.sid) {
-            throw new RecordFormatException("Unexpected header-footer record sid: 0x" + Integer.toHexString(rec.getSid()));
+            throw new org.apache.poi.util.RecordFormatException("Unexpected header-footer record sid: 0x" + Integer.toHexString(rec.getSid()));
         }
         _headerFooter = rec;
     }
@@ -647,7 +647,7 @@ public final class PageSettingsBlock extends RecordAggregate {
      *
      * @param rs the RecordStream to read from
      * 
-     * @throws  RecordFormatException if any PSB record encountered has the same type (sid) as
+     * @throws  org.apache.poi.util.RecordFormatException if any PSB record encountered has the same type (sid) as
      * a record that is already part of this PageSettingsBlock
      */
     public void addLateRecords(RecordStream rs) {
index e6f5b91d5640999c6ddeb50dfe43cf54ee2c9582..2bbc18cd17e85c1f1436147e78d82bc6f3e7d545 100644 (file)
@@ -66,8 +66,7 @@ public final class HSSFCellStyle implements CellStyle {
      *  unless you're comparing which one is which)
      */
     @Override
-    public short getIndex()
-    {
+    public short getIndex() {
         return _index;
     }
 
@@ -489,8 +488,7 @@ public final class HSSFCellStyle implements CellStyle {
     @Deprecated
     @Removal(version="3.17")
     @Override
-    public void setBorderLeft(short border)
-    {
+    public void setBorderLeft(short border) {
         _format.setIndentNotParentBorder(true);
         _format.setBorderLeft(border);
     }
index 129066fa1c3a6a1876aca8447fe2416c15aa1ecd..75e143c1c702c0752fbe24455e4df681152d20b0 100644 (file)
@@ -48,7 +48,7 @@ public class HSSFColor implements Color {
 
     /**
      * Predefined HSSFColors with their given palette index (and an optional 2nd index)
-     * 
+     *
      * @since POI 3.16 beta 2
      */
     public enum HSSFColorPredefined {
@@ -140,7 +140,7 @@ public class HSSFColor implements Color {
         public String getHexString() {
             return color.getHexString();
         }
-        
+
         /**
          * @return (a copy of) the HSSFColor assigned to the enum
          */
@@ -211,7 +211,7 @@ public class HSSFColor implements Color {
      *
      * @return a Map containing all colors keyed by String gnumeric-like triplets
      */
-    public final static Map<String,HSSFColor> getTripletHash()
+    public static Map<String,HSSFColor> getTripletHash()
     {
         return createColorsByHexStringMap();
     }
@@ -231,7 +231,7 @@ public class HSSFColor implements Color {
 
     /**
      * Maps the Enums to the HSSFColor subclasses, in cases of user code evaluating the classname
-     * 
+     *
      * @deprecated in 3.16 - remove mapping when subclasses are removed and access
      *  HSSFColorPredfined.values() directly (but exclude AUTOMATIC)
      */
index e191f5c7d99c9efc83da9690f1a9a042d4e9e8d7..97b3eca3e68f182e601d0767654cb2522b07af0a 100644 (file)
@@ -28,7 +28,7 @@ import java.util.ArrayList;
 public class HexRead {
     /**
      * This method reads hex data from a filename and returns a byte array.
-     * The file may contain line comments that are preceeded with a # symbol.
+     * The file may contain line comments that are preceded with a # symbol.
      *
      * @param filename  The filename to read
      * @return The bytes read from the file.
index c5d2ba325ca12a09e66a8934e1d89378ee96db0a..7706ca19e34befdb6ae2da3c88c7c5318a14035b 100644 (file)
@@ -21,10 +21,7 @@ import java.awt.Color;
 import java.awt.Graphics2D;
 import java.awt.RenderingHints;
 import java.awt.image.BufferedImage;
-import java.io.File;
-import java.io.FileInputStream;
-import java.io.FileOutputStream;
-import java.io.IOException;
+import java.io.*;
 
 import javax.imageio.ImageIO;
 
@@ -94,7 +91,7 @@ public class VsdxToPng {
 
         graphics.dispose();
 
-        FileOutputStream out = new FileOutputStream(outFile);
+        OutputStream out = new FileOutputStream(outFile);
         try {
             ImageIO.write(img, "png", out);
         } finally {
index 026a45be7a69af9bd64da5f2d9429d0ef6b1995d..551bf405ba92b33414ae65fc6bc9d0bbceede827 100644 (file)
@@ -107,7 +107,7 @@ public class SXSSFWorkbook implements Workbook {
     /**
      * whether temp files should be compressed.
      */
-    private boolean _compressTmpFiles = false;
+    private boolean _compressTmpFiles;
 
     /**
      * shared string table - a cache of strings in this workbook
@@ -817,6 +817,7 @@ public class SXSSFWorkbook implements Workbook {
     @Removal(version="3.17")
     public Font findFont(short boldWeight, short color, short fontHeight, String name, boolean italic, boolean strikeout, short typeOffset, byte underline)
     {
+        //noinspection deprecation
         return _wb.findFont(boldWeight, color, fontHeight, name, italic, strikeout, typeOffset, underline);
     }
     
@@ -1027,8 +1028,8 @@ public class SXSSFWorkbook implements Workbook {
     @Override
     @Deprecated
     @Removal(version="3.18")
-    public Name getNameAt(int nameIndex)
-    {
+    public Name getNameAt(int nameIndex) {
+        //noinspection deprecation
         return _wb.getNameAt(nameIndex);
     }
 
@@ -1058,8 +1059,8 @@ public class SXSSFWorkbook implements Workbook {
     @Override
     @Deprecated
     @Removal(version="3.18")
-    public int getNameIndex(String name)
-    {
+    public int getNameIndex(String name) {
+        //noinspection deprecation
         return _wb.getNameIndex(name);
     }
 
@@ -1073,8 +1074,8 @@ public class SXSSFWorkbook implements Workbook {
     @Override
     @Deprecated
     @Removal(version="3.18")
-    public void removeName(int index)
-    {
+    public void removeName(int index) {
+        //noinspection deprecation
         _wb.removeName(index);
     }
 
@@ -1088,8 +1089,8 @@ public class SXSSFWorkbook implements Workbook {
     @Override
     @Deprecated
     @Removal(version="3.18")
-    public void removeName(String name)
-    {
+    public void removeName(String name) {
+        //noinspection deprecation
         _wb.removeName(name);
     }
 
@@ -1298,8 +1299,12 @@ public class SXSSFWorkbook implements Workbook {
      *  formulas such as "[MyOtherWorkbook]Sheet3!$A$5" to be added to the 
      *  file, for workbooks not already referenced.
      *
+     *  Note: this is not implemented and thus currently throws an Exception stating this.
+     *
      * @param name The name the workbook will be referenced as in formulas
      * @param workbook The open workbook to fetch the link required information from
+     *
+     * @throws RuntimeException stating that this method is not implemented yet.
      */
     @Override
     @NotImplemented
index e67ca1be4e8b2bd213fdb33213b8cd4ed23368ef..fecad97bc8c002f0ce53dcc710ef77ffaa243f45 100644 (file)
@@ -2076,8 +2076,12 @@ public class XSSFWorkbook extends POIXMLDocument implements Workbook {
      *  formulas such as "[MyOtherWorkbook.xlsx]Sheet3!$A$5" to be added to the 
      *  file, for workbooks not already linked / referenced.
      *
+     *  Note: this is not implemented and thus currently throws an Exception stating this.
+     *
      * @param name The name the workbook will be referenced as in formulas
      * @param workbook The open workbook to fetch the link required information from
+     *
+     * @throws RuntimeException stating that this method is not implemented yet.
      */
     @Override
     @NotImplemented
index 02b2dbdaa3dc56efc4e50bf242a5ee8193a38f9b..e370e5750699344a9758c743627f24f77a5207fb 100644 (file)
@@ -1003,7 +1003,6 @@ public abstract class AbstractWordConverter
 
                 processEndnoteAutonumbered( doc, noteIndex, block,
                         noteTextRange );
-                return;
             }
         }
     }
index dcb40b6932c643153e6e40fd35663c7af4e818ca..1a6d8b8f4048512dceb326585cc990be2fbf9196 100644 (file)
@@ -158,7 +158,7 @@ public class WordToHtmlConverter extends AbstractWordConverter
 
     private final HtmlDocumentFacade htmlDocumentFacade;
 
-    private Element notes = null;
+    private Element notes;
 
     /**
      * Creates new instance of {@link WordToHtmlConverter}. Can be used for
index e89efe7354abdc0796b72a150fa023c5a5ef782b..65ec6b9178044009b4412a5b731714df1cefa093 100644 (file)
@@ -392,8 +392,7 @@ public final class HWPFLister
             for ( char c : text.toCharArray() )
             {
                 if ( c < 30 )
-                    stringBuilder
-                            .append( "\\0x" + Integer.toHexString( c ) );
+                    stringBuilder.append("\\0x").append(Integer.toHexString(c));
                 else
                     stringBuilder.append( c );
             }
index d5dc0689243673c3f13ef6dc027285887386722d..a894f4ed91ac8de1d4c969db137916a1b3be705d 100644 (file)
@@ -96,7 +96,7 @@ public final class WordExtractor extends POIOLE2TextExtractor
         }
 
         // Process the first argument as a file
-        FileInputStream fin = new FileInputStream( args[0] );
+        InputStream fin = new FileInputStream( args[0] );
         WordExtractor extractor = new WordExtractor( fin );
         try {
                System.out.println( extractor.getText() );
index ed06d35506b7ed3264253b23b6b36f7bf5f63494..a7ed1adc4b42b4d3920fa9bf1dce37939259f745 100644 (file)
@@ -42,15 +42,16 @@ import org.apache.poi.util.StringUtil;
 /**
  * Represents embedded picture extracted from Word Document
  */
-public final class Picture
-{
+public final class Picture {
+    private static final POILogger log = POILogFactory
+            .getLogger( Picture.class );
+
     public static final byte[] COMPRESSED1 = { (byte) 0xFE, 0x78, (byte) 0xDA };
 
     public static final byte[] COMPRESSED2 = { (byte) 0xFE, 0x78, (byte) 0x9C };
 
     public static final byte[] IHDR = new byte[] { 'I', 'H', 'D', 'R' };
-    private static final POILogger log = POILogFactory
-            .getLogger( Picture.class );
+
     @Deprecated
     private static final byte[] PNG = new byte[] { (byte) 0x89, 0x50, 0x4E,
             0x47, 0x0D, 0x0A, 0x1A, 0x0A };
@@ -401,10 +402,11 @@ public final class Picture
     }
 
     /**
-     * Returns picture's content as it stored in Word file, i.e. possibly in
+     * Returns picture's content as stored in the Word file, i.e. possibly in
      * compressed form.
      * 
-     * @return picture's content as it stored in Word file
+     * @return picture's content as it stored in Word file or an empty byte array
+     *      if it cannot be read.
      */
     public byte[] getRawContent()
     {
index 1b18e5fd3a9415461b6722636d03486d21489d58..b2f4082999d61fbe566ea0f3298bf9e86061b1c9 100644 (file)
@@ -82,8 +82,7 @@ public class TestWordToHtmlConverter
                 new DOMSource( wordToHtmlConverter.getDocument() ),
                 new StreamResult( stringWriter ));
 
-        String result = stringWriter.toString();
-        return result;
+        return stringWriter.toString();
     }
 
     @Test
index a63fd7b03907ec4522d6271413925891e9c6b7a0..0f6caa758981ed67eafdea8ab9d19842fc70096b 100644 (file)
@@ -203,7 +203,6 @@ public final class TestPageSettingsBlock extends TestCase {
         * was added while fixing bug 47199.  All existing POI test samples comply with this requirement.
         */
        public void testDuplicatePSBRecord_bug47199() {
-
                // Hypothetical setup of PSB records which should cause POI to crash
                Record[] recs = {
                                new HeaderRecord("&LSales Figures"),
@@ -214,10 +213,8 @@ public final class TestPageSettingsBlock extends TestCase {
                try {
                        new PageSettingsBlock(rs);
                        throw new AssertionFailedError("Identified bug 47199b - duplicate PSB records should not be allowed");
-               } catch (RecordFormatException e) {
-                       if (e.getMessage().equals("Duplicate PageSettingsBlock record (sid=0x14)")) {
-                               // expected during successful test
-                       } else {
+               } catch (org.apache.poi.util.RecordFormatException e) {
+                       if (!e.getMessage().equals("Duplicate PageSettingsBlock record (sid=0x14)")) {
                                throw new AssertionFailedError("Expected RecordFormatException due to duplicate PSB record");
                        }
                }
@@ -288,7 +285,7 @@ public final class TestPageSettingsBlock extends TestCase {
                PageSettingsBlock psb;
                try {
                        psb = new PageSettingsBlock(rs);
-               } catch (RecordFormatException e) {
+               } catch (org.apache.poi.util.RecordFormatException e) {
                        if ("Duplicate PageSettingsBlock record (sid=0x4d)".equals(e.getMessage())) {
                                throw new AssertionFailedError("Identified bug 47415");
                        }