]> source.dussan.org Git - poi.git/commitdiff
FindBugs fix
authorAndreas Beeker <kiwiwings@apache.org>
Thu, 13 Feb 2014 23:34:11 +0000 (23:34 +0000)
committerAndreas Beeker <kiwiwings@apache.org>
Thu, 13 Feb 2014 23:34:11 +0000 (23:34 +0000)
- fixed "Field isn't final but should be"
- see http://findbugs.sourceforge.net/bugDescriptions.html#MS_SHOULD_BE_FINAL

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

24 files changed:
build.xml
src/java/org/apache/poi/hpsf/Thumbnail.java
src/java/org/apache/poi/hpsf/VariantSupport.java
src/java/org/apache/poi/hssf/record/FtCfSubRecord.java
src/java/org/apache/poi/hssf/record/FtPioGrbitSubRecord.java
src/java/org/apache/poi/hssf/record/LbsDataSubRecord.java
src/ooxml/java/org/apache/poi/extractor/CommandLineTextExtractor.java
src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFRelation.java
src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFRelation.java
src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFRelation.java
src/resources/devtools/findbugs-filters.xml
src/scratchpad/src/org/apache/poi/hslf/dev/PPTXMLDump.java
src/scratchpad/src/org/apache/poi/hslf/model/AutoShapes.java
src/scratchpad/src/org/apache/poi/hslf/model/ShapeFactory.java
src/scratchpad/src/org/apache/poi/hslf/model/ShapePainter.java
src/scratchpad/src/org/apache/poi/hslf/model/ShapeTypes.java
src/scratchpad/src/org/apache/poi/hslf/model/textproperties/CharFlagsTextProp.java
src/scratchpad/src/org/apache/poi/hslf/model/textproperties/ParagraphFlagsTextProp.java
src/scratchpad/src/org/apache/poi/hslf/record/Record.java
src/scratchpad/src/org/apache/poi/hslf/record/RecordTypes.java
src/scratchpad/src/org/apache/poi/hslf/record/SSSlideInfoAtom.java
src/scratchpad/src/org/apache/poi/hslf/record/StyleTextPropAtom.java
src/scratchpad/src/org/apache/poi/hslf/usermodel/PictureData.java
src/scratchpad/src/org/apache/poi/hwpf/model/PropertyNode.java

index 4500454aef8b657fdb2a7d9f9155b48d5a573403..914eecc02078b1ce10bc3664909cdf3eb3b9b13c 100644 (file)
--- a/build.xml
+++ b/build.xml
@@ -1405,7 +1405,7 @@ under the License.
                        </classpath>
                </taskdef>
                <findbugs home="${findbugs.home}" output="html" outputFile="build/findbugs.html"
-                       includeFilter="src/resources/devtools/findbugs-filters.xml">
+                       excludeFilter="src/resources/devtools/findbugs-filters.xml">
                        <fileset dir="${dist.dir}">
                                <include name="poi-${version.id}-*.jar"/>
                                <include name="poi-scratchpad-${version.id}-*.jar"/>
index c39abe0a9eb16c1de48c7431bd7bc58905d42ee8..8d8ba7d20974f434a8d61fcaa2f58a0bcf4746e7 100644 (file)
@@ -32,7 +32,7 @@ public final class Thumbnail {
      * <code>byte[]</code> returned by {@link
      * SummaryInformation#getThumbnail()}</p>
      */
-    public static int OFFSET_CFTAG = 4;
+    public static final int OFFSET_CFTAG = 4;
 
     /**
      * <p>Offset in bytes where the Clipboard Format starts in the
@@ -42,7 +42,7 @@ public final class Thumbnail {
      * <p>This is only valid if the Clipboard Format Tag is {@link
      * #CFTAG_WINDOWS}</p>
      */
-    public static int OFFSET_CF = 8;
+    public static final int OFFSET_CF = 8;
 
     /**
      * <p>Offset in bytes where the Windows Metafile (WMF) image data
@@ -60,7 +60,7 @@ public final class Thumbnail {
      * image. It can be saved to disk with a <code>.wmf</code> file
      * type and read using a WMF-capable image viewer.</p>
      */
-    public static int OFFSET_WMFDATA = 20;
+    public static final int OFFSET_WMFDATA = 20;
 
     /**
      * <p>Clipboard Format Tag - Windows clipboard format</p>
@@ -68,7 +68,7 @@ public final class Thumbnail {
      * <p>A <code>DWORD</code> indicating a built-in Windows clipboard
      * format value</p>
      */
-    public static int CFTAG_WINDOWS = -1;
+    public static final int CFTAG_WINDOWS = -1;
 
     /**
      * <p>Clipboard Format Tag - Macintosh clipboard format</p>
@@ -76,7 +76,7 @@ public final class Thumbnail {
      * <p>A <code>DWORD</code> indicating a Macintosh clipboard format
      * value</p>
      */
-    public static int CFTAG_MACINTOSH = -2;
+    public static final int CFTAG_MACINTOSH = -2;
 
     /**
      * <p>Clipboard Format Tag - Format ID</p>
@@ -84,7 +84,7 @@ public final class Thumbnail {
      * <p>A GUID containing a format identifier (FMTID). This is
      * rarely used.</p>
      */
-    public static int CFTAG_FMTID = -3;
+    public static final int CFTAG_FMTID = -3;
 
     /**
      * <p>Clipboard Format Tag - No Data</p>
@@ -92,7 +92,7 @@ public final class Thumbnail {
      * <p>A <code>DWORD</code> indicating No data. This is rarely
      * used.</p>
      */
-    public static int CFTAG_NODATA = 0;
+    public static final int CFTAG_NODATA = 0;
 
     /**
      * <p>Clipboard Format - Windows metafile format. This is the
@@ -102,17 +102,17 @@ public final class Thumbnail {
      * regular WMF images. The clipboard version of this format has an
      * extra clipboard-specific header.</p>
      */
-    public static int CF_METAFILEPICT = 3;
+    public static final int CF_METAFILEPICT = 3;
 
     /**
      * <p>Clipboard Format - Device Independent Bitmap</p>
      */
-    public static int CF_DIB = 8;
+    public static final int CF_DIB = 8;
 
     /**
      * <p>Clipboard Format - Enhanced Windows metafile format</p>
      */
-    public static int CF_ENHMETAFILE = 14;
+    public static final int CF_ENHMETAFILE = 14;
 
     /**
      * <p>Clipboard Format - Bitmap</p>
@@ -121,7 +121,7 @@ public final class Thumbnail {
      * href="msdn.microsoft.com/library/en-us/dnw98bk/html/clipboardoperations.asp
      * target="_blank">msdn.microsoft.com/library/en-us/dnw98bk/html/clipboardoperations.asp</a>.</p>
      */
-    public static int CF_BITMAP = 2;
+    public static final int CF_BITMAP = 2;
 
     /**
      * <p>A <code>byte[]</code> to hold a thumbnail image in ({@link
index 44d4ad76cbd7f5b437e374d38b30a0c2da42dd78..fe327b3d99045a7d9c2f36be431e0a8f3e725b0e 100644 (file)
@@ -49,7 +49,7 @@ import org.apache.poi.util.POILogger;
  */
 public class VariantSupport extends Variant
 {
-       private static POILogger logger = POILogFactory.getLogger(VariantSupport.class);
+       private static final POILogger logger = POILogFactory.getLogger(VariantSupport.class);
     private static boolean logUnsupportedTypes = false;
 
     /**
index 95438334e11b0c256cf73f1023b2daea36a1b21a..1c85e9550fe019e3790e06bd140c67fb87cdcacb 100644 (file)
@@ -32,18 +32,18 @@ public final class FtCfSubRecord extends SubRecord {
     /**
      * Specifies the format of the picture is an enhanced metafile.
      */
-    public static short METAFILE_BIT    = (short)0x0002;
+    public static final short METAFILE_BIT    = (short)0x0002;
 
     /**
      * Specifies the format of the picture is a bitmap.
      */
-    public static short BITMAP_BIT      = (short)0x0009;
+    public static final short BITMAP_BIT      = (short)0x0009;
     
     /**
      * Specifies the picture is in an unspecified format that is
      * neither and enhanced metafile nor a bitmap.
      */
-    public static short UNSPECIFIED_BIT = (short)0xFFFF;
+    public static final short UNSPECIFIED_BIT = (short)0xFFFF;
     
     private short flags = 0;
 
index 8562a0535248db02d555e437b1a08f0f845e384b..068d17a24adbfa572b09dd97fa99226a4eeed1a7 100644 (file)
@@ -33,53 +33,53 @@ public final class FtPioGrbitSubRecord extends SubRecord {
      * A bit that specifies whether the picture's aspect ratio is preserved when rendered in 
      * different views (Normal view, Page Break Preview view, Page Layout view and printing).
      */
-    public static int AUTO_PICT_BIT    = 1 << 0;
+    public static final int AUTO_PICT_BIT    = 1 << 0;
 
     /**
      * A bit that specifies whether the pictFmla field of the Obj record that contains 
      * this FtPioGrbit specifies a DDE reference.
      */
-    public static int DDE_BIT          = 1 << 1;
+    public static final int DDE_BIT          = 1 << 1;
     
     /**
      * A bit that specifies whether this object is expected to be updated on print to
      * reflect the values in the cell associated with the object.
      */
-    public static int PRINT_CALC_BIT   = 1 << 2;
+    public static final int PRINT_CALC_BIT   = 1 << 2;
 
     /**
      * A bit that specifies whether the picture is displayed as an icon.
      */
-    public static int ICON_BIT         = 1 << 3;
+    public static final int ICON_BIT         = 1 << 3;
     
     /**
      * A bit that specifies whether this object is an ActiveX control.
      * It MUST NOT be the case that both fCtl and fDde are equal to 1.
      */
-    public static int CTL_BIT          = 1 << 4;
+    public static final int CTL_BIT          = 1 << 4;
     
     /**
      * A bit that specifies whether the object data are stored in an
      * embedding storage (= 0) or in the controls stream (ctls) (= 1).
      */
-    public static int PRSTM_BIT        = 1 << 5;
+    public static final int PRSTM_BIT        = 1 << 5;
     
     /**
      * A bit that specifies whether this is a camera picture.
      */
-    public static int CAMERA_BIT       = 1 << 7;
+    public static final int CAMERA_BIT       = 1 << 7;
     
     /**
      * A bit that specifies whether this picture's size has been explicitly set.
      * 0 = picture size has been explicitly set, 1 = has not been set
      */
-    public static int DEFAULT_SIZE_BIT = 1 << 8;
+    public static final int DEFAULT_SIZE_BIT = 1 << 8;
     
     /**
      * A bit that specifies whether the OLE server for the object is called
      * to load the object's data automatically when the parent workbook is opened.
      */
-    public static int AUTO_LOAD_BIT    = 1 << 9;
+    public static final int AUTO_LOAD_BIT    = 1 << 9;
 
     
     private short flags = 0;
index c6255e7d3f6f3296008efd0c6cc36f3ef6abd96d..93a0430eb5b184c3c5c017ce9b917cdb9c5658b9 100644 (file)
 ==================================================================== */\r
 package org.apache.poi.hssf.record;\r
 \r
-import org.apache.poi.ss.formula.ptg.*;\r
-import org.apache.poi.util.*;\r
+import org.apache.poi.ss.formula.ptg.Ptg;\r
+import org.apache.poi.util.HexDump;\r
+import org.apache.poi.util.LittleEndianInput;\r
+import org.apache.poi.util.LittleEndianOutput;\r
+import org.apache.poi.util.StringUtil;\r
 \r
 /**\r
  * This structure specifies the properties of a list or drop-down list embedded object in a sheet.\r
@@ -298,15 +301,15 @@ public class LbsDataSubRecord extends SubRecord {
         /**\r
          * Combo dropdown control\r
          */\r
-        public static int STYLE_COMBO_DROPDOWN = 0;\r
+        public static final int STYLE_COMBO_DROPDOWN = 0;\r
         /**\r
          * Combo Edit dropdown control\r
          */\r
-        public static int STYLE_COMBO_EDIT_DROPDOWN = 1;\r
+        public static final int STYLE_COMBO_EDIT_DROPDOWN = 1;\r
         /**\r
          * Simple dropdown control (just the dropdown button)\r
          */\r
-        public static int STYLE_COMBO_SIMPLE_DROPDOWN = 2;\r
+        public static final int STYLE_COMBO_SIMPLE_DROPDOWN = 2;\r
 \r
         /**\r
          *  An unsigned integer that specifies the style of this dropdown. \r
index 76cb44d4fbc82735ad1eaf92470c4901cf0c84e8..8f9ea7599e67003624625284d00c9477a5353941 100644 (file)
@@ -25,7 +25,7 @@ import org.apache.poi.POITextExtractor;
  *  for when debugging.
  */
 public class CommandLineTextExtractor {
-   public static String DIVIDER = "=======================";
+   public static final String DIVIDER = "=======================";
    
    public static void main(String[] args) throws Exception {
       if(args.length < 1) {
index 046e71e6a57a46b127ff37dcf499c036d1c81caf..82e073167f6168bb36b6e36c9addfe5bf44078f7 100644 (file)
 ==================================================================== */
 package org.apache.poi.xslf.usermodel;
 
+import java.util.HashMap;
+import java.util.Map;
+
 import org.apache.poi.POIXMLDocumentPart;
 import org.apache.poi.POIXMLRelation;
 import org.apache.poi.util.Beta;
 import org.apache.poi.util.POILogFactory;
 import org.apache.poi.util.POILogger;
 
-import java.util.HashMap;
-import java.util.Map;
-
 @Beta
 public class XSLFRelation extends POIXMLRelation {
 
-   private static POILogger log = POILogFactory.getLogger(XSLFRelation.class);
+   private static final POILogger log = POILogFactory.getLogger(XSLFRelation.class);
 
    /**
     * A map to lookup POIXMLRelation by its relation type
     */
-   protected static Map<String, XSLFRelation> _table = new HashMap<String, XSLFRelation>();
+   protected static final Map<String, XSLFRelation> _table = new HashMap<String, XSLFRelation>();
    
    public static final XSLFRelation MAIN = new XSLFRelation(
            "application/vnd.openxmlformats-officedocument.presentationml.presentation.main+xml",
index bdb1ea818b7b2c2a1ce9e306e9f7526d6003f71b..037c41356f03437970b1b25b3a5f8684b0a440c4 100644 (file)
@@ -47,12 +47,12 @@ import org.apache.poi.xssf.model.ThemesTable;
  */
 public final class XSSFRelation extends POIXMLRelation {
 
-    private static POILogger log = POILogFactory.getLogger(XSSFRelation.class);
+    private static final POILogger log = POILogFactory.getLogger(XSSFRelation.class);
 
     /**
      * A map to lookup POIXMLRelation by its relation type
      */
-    protected static Map<String, XSSFRelation> _table = new HashMap<String, XSSFRelation>();
+    protected static final Map<String, XSSFRelation> _table = new HashMap<String, XSSFRelation>();
 
 
     public static final XSSFRelation WORKBOOK = new XSSFRelation(
index 1ff2bd89438abf1f96028c0c590a91912bd3a3ef..e20052b68dc96f3fbe1aa41261e7a285f80f13c8 100644 (file)
@@ -31,7 +31,7 @@ public final class XWPFRelation extends POIXMLRelation {
     /**
      * A map to lookup POIXMLRelation by its relation type
      */
-    protected static Map<String, XWPFRelation> _table = new HashMap<String, XWPFRelation>();
+    protected static final Map<String, XWPFRelation> _table = new HashMap<String, XWPFRelation>();
 
 
     public static final XWPFRelation DOCUMENT = new XWPFRelation(
index 4b47561d697397d3a0401a74da6a18b75202f0e4..9ec102bfd9f06c207c7a51f59e15ab5bf2664394 100644 (file)
@@ -18,7 +18,7 @@
    ====================================================================\r
 -->\r
 <FindBugsFilter>\r
-<Match>\r
-<Not><Bug pattern="CN_IMPLEMENTS_CLONE_BUT_NOT_CLONEABLE"/></Not>\r
-</Match>\r
+       <Match>\r
+               <Bug pattern="CN_IMPLEMENTS_CLONE_BUT_NOT_CLONEABLE,MS_PKGPROTECT,MS_MUTABLE_ARRAY"/>\r
+       </Match>\r
 </FindBugsFilter>
\ No newline at end of file
index 4b2c9b913b7516ff9f1f9dde05c4c51f49a3316f..0b57552619fdf5e8fd313a9dfb40ad9dcce98539 100644 (file)
 
 package org.apache.poi.hslf.dev;
 
-import org.apache.poi.util.LittleEndian;
+import java.io.File;
+import java.io.FileInputStream;
+import java.io.FileNotFoundException;
+import java.io.FileWriter;
+import java.io.IOException;
+import java.io.StringWriter;
+import java.io.Writer;
+
 import org.apache.poi.hslf.record.RecordTypes;
-import org.apache.poi.poifs.filesystem.*;
-import java.io.*;
+import org.apache.poi.poifs.filesystem.DocumentEntry;
+import org.apache.poi.poifs.filesystem.DocumentInputStream;
+import org.apache.poi.poifs.filesystem.POIFSFileSystem;
+import org.apache.poi.util.LittleEndian;
 
 /**
  * Utility class which dumps raw contents of a ppt file into XML format
@@ -33,7 +42,7 @@ public final class PPTXMLDump {
     public static final int PICT_HEADER_SIZE = 25; //size of the picture header
     public final static String PPDOC_ENTRY = "PowerPoint Document";
     public final static String PICTURES_ENTRY = "Pictures";
-    public static String CR = System.getProperty("line.separator");
+    public final static String CR = System.getProperty("line.separator");
 
     protected Writer out;
     protected byte[] docstream;
index bc1cc1293f98473d6f966d272fca22e1d2a196a0..02b20fbe2a7b538d7a8e8118988a3dc6f3b82f1f 100644 (file)
 
 package org.apache.poi.hslf.model;
 
-import org.apache.poi.ddf.EscherProperties;
+import java.awt.geom.AffineTransform;
+import java.awt.geom.Arc2D;
+import java.awt.geom.Ellipse2D;
+import java.awt.geom.GeneralPath;
+import java.awt.geom.Line2D;
+import java.awt.geom.Rectangle2D;
+import java.awt.geom.RoundRectangle2D;
 
-import java.awt.geom.*;
+import org.apache.poi.ddf.EscherProperties;
 
 /**
  * Stores definition of auto-shapes.
@@ -30,7 +36,7 @@ import java.awt.geom.*;
  * @author Yegor Kozlov
  */
 public final class AutoShapes {
-    protected static ShapeOutline[] shapes;
+       protected static final ShapeOutline[] shapes;
 
 
     /**
index 464c842348163da49b10b931747b045d34550d90..215835d5be8d267fbd1b2b7ceead3e436feb72aa 100644 (file)
 
 package org.apache.poi.hslf.model;
 
-import org.apache.poi.ddf.*;
-import org.apache.poi.util.POILogger;
-import org.apache.poi.util.POILogFactory;
-import org.apache.poi.hslf.record.*;
-
-import java.util.List;
 import java.util.Iterator;
+import java.util.List;
+
+import org.apache.poi.ddf.EscherClientDataRecord;
+import org.apache.poi.ddf.EscherContainerRecord;
+import org.apache.poi.ddf.EscherOptRecord;
+import org.apache.poi.ddf.EscherProperties;
+import org.apache.poi.ddf.EscherProperty;
+import org.apache.poi.ddf.EscherPropertyFactory;
+import org.apache.poi.ddf.EscherRecord;
+import org.apache.poi.ddf.EscherSimpleProperty;
+import org.apache.poi.ddf.EscherSpRecord;
+import org.apache.poi.hslf.record.InteractiveInfo;
+import org.apache.poi.hslf.record.InteractiveInfoAtom;
+import org.apache.poi.hslf.record.OEShapeAtom;
+import org.apache.poi.hslf.record.Record;
+import org.apache.poi.hslf.record.RecordTypes;
+import org.apache.poi.util.POILogFactory;
+import org.apache.poi.util.POILogger;
 
 /**
  * Create a <code>Shape</code> object depending on its type
@@ -32,7 +44,7 @@ import java.util.Iterator;
  */
 public final class ShapeFactory {
     // For logging
-    protected static POILogger logger = POILogFactory.getLogger(ShapeFactory.class);
+    protected static final POILogger logger = POILogFactory.getLogger(ShapeFactory.class);
 
     /**
      * Create a new shape from the data provided.
index 09474a0b8c8cf1afa6c8c294da49f13e21c4d461..c9954a8dee2d38d5e1d2b3993f74dbaabebe86d4 100644 (file)
 package org.apache.poi.hslf.model;
 
 
-import org.apache.poi.util.POILogger;
-import org.apache.poi.util.POILogFactory;
-
-import java.awt.*;
+import java.awt.BasicStroke;
+import java.awt.Color;
+import java.awt.Graphics2D;
+import java.awt.Stroke;
 import java.awt.geom.Rectangle2D;
 
+import org.apache.poi.util.POILogFactory;
+import org.apache.poi.util.POILogger;
+
 /**
  * Paint a shape into java.awt.Graphics2D
  *
  * @author Yegor Kozlov
  */
 public final class ShapePainter {
-    protected static POILogger logger = POILogFactory.getLogger(ShapePainter.class);
+    protected static final POILogger logger = POILogFactory.getLogger(ShapePainter.class);
 
     public static void paint(SimpleShape shape, Graphics2D graphics){
         Rectangle2D anchor = shape.getLogicalAnchor2D();
index 7b5a17deccc1aeb005c6390835be0ba52306ca2f..4840c9313082604a10e015d447e739682f3347f2 100644 (file)
 
 package org.apache.poi.hslf.model;
 
-import org.apache.poi.hslf.exceptions.HSLFException;
-
-import java.util.HashMap;
 import java.lang.reflect.Field;
+import java.util.HashMap;
+
+import org.apache.poi.hslf.exceptions.HSLFException;
 
 /**
  * Contains all known shape types in PowerPoint
@@ -38,7 +38,7 @@ public final class ShapeTypes implements org.apache.poi.sl.usermodel.ShapeTypes
         return name;
     }
 
-    public static HashMap types;
+    public static final HashMap types;
     static {
         types = new HashMap();
         try {
index f038781534b92c7ddad72861ef0b71294aeec6cb..1afd6b1810ce2a7b30ce69dfb1a6c2424aac3c9b 100644 (file)
@@ -32,7 +32,7 @@ public class CharFlagsTextProp extends BitMaskTextProp {
        public static final int ENABLE_NUMBERING_1_IDX = 11;
        public static final int ENABLE_NUMBERING_2_IDX = 12;
 
-    public static String NAME = "char_flags";
+    public static final String NAME = "char_flags";
        public CharFlagsTextProp() {
                super(2,0xffff, NAME, new String[] {
                                "bold",                 // 0x0001  A bit that specifies whether the characters are bold.
index f1bce9593a1243c11bf4611848de2ef91402228f..c0501d2f9054ea4f37187eff40e2962d2a79caee 100644 (file)
@@ -28,7 +28,7 @@ public final class ParagraphFlagsTextProp extends BitMaskTextProp {
        public static final int BULLET_HARDCOLOR_IDX = 2;
        public static final int BULLET_HARDSIZE_IDX = 4;
 
-    public static String NAME = "paragraph_flags";
+    public static final String NAME = "paragraph_flags";
 
        public ParagraphFlagsTextProp() {
                super(2,  0xF, NAME, new String[] {
index 5530e2454d3f879ff84996acde798e87504b8958..eb6944ac69a5dbc124c22fc1cffeff269d7eb745 100644 (file)
@@ -38,7 +38,7 @@ import org.apache.poi.util.POILogger;
 public abstract class Record
 {
     // For logging
-       protected static POILogger logger = POILogFactory.getLogger(Record.class);
+       protected static final POILogger logger = POILogFactory.getLogger(Record.class);
 
        /**
         * Is this record type an Atom record (only has data),
index 2b2dc4bddf4a8bb96792345f76e74f50de1d5cdb..51b8f26a89f2112c321d66908d00821b9d289d6b 100644 (file)
@@ -17,8 +17,8 @@
 
 package org.apache.poi.hslf.record;
 
-import java.util.HashMap;
 import java.lang.reflect.Field;
+import java.util.HashMap;
 
 /**
  * List of all known record types in a PowerPoint document, and the
@@ -31,8 +31,8 @@ import java.lang.reflect.Field;
  * @author Nick Burch
  */
 public final class RecordTypes {
-    public static HashMap<Integer,String> typeToName;
-    public static HashMap<Integer,Class<? extends Record>> typeToClass;
+    public static final HashMap<Integer,String> typeToName;
+    public static final HashMap<Integer,Class<? extends Record>> typeToClass;
 
     public static final Type Unknown = new Type(0,null);
     public static final Type Document = new Type(1000,Document.class);
index 0fbc7ff75f2e99f87bfd860116484d004c4f3d4b..931f5dc70a87b50a4b6695c49b3aa8afd82586be 100644 (file)
@@ -64,42 +64,42 @@ public class SSSlideInfoAtom extends RecordAtom {
      * A bit that specifies whether the presentation slide can be\r
      * manually advanced by the user during the slide show.\r
      */\r
-    public static int MANUAL_ADVANCE_BIT     = 1 << 0;\r
+    public static final int MANUAL_ADVANCE_BIT     = 1 << 0;\r
     \r
     /**\r
      * A bit that specifies whether the corresponding slide is \r
      * hidden and is not displayed during the slide show.\r
      */\r
-    public static int HIDDEN_BIT             = 1 << 2;\r
+    public static final int HIDDEN_BIT             = 1 << 2;\r
     \r
     /**\r
      * A bit that specifies whether to play the sound specified by soundIfRef.\r
      */\r
-    public static int SOUND_BIT              = 1 << 4;\r
+    public static final int SOUND_BIT              = 1 << 4;\r
     \r
     /**\r
      * A bit that specifies whether the sound specified by soundIdRef is\r
      * looped continuously when playing until the next sound plays.\r
      */\r
-    public static int LOOP_SOUND_BIT         = 1 << 6;\r
+    public static final int LOOP_SOUND_BIT         = 1 << 6;\r
     \r
     /**\r
      * A bit that specifies whether to stop any currently playing \r
      * sound when the transition starts.\r
      */\r
-    public static int STOP_SOUND_BIT         = 1 << 8;\r
+    public static final int STOP_SOUND_BIT         = 1 << 8;\r
     \r
     /**\r
      * A bit that specifies whether the slide will automatically\r
      * advance after slideTime milliseconds during the slide show.\r
      */\r
-    public static int AUTO_ADVANCE_BIT       = 1 << 10;\r
+    public static final int AUTO_ADVANCE_BIT       = 1 << 10;\r
 \r
     /**\r
      * A bit that specifies whether to display the cursor during\r
      * the slide show. \r
      */\r
-    public static int CURSOR_VISIBLE_BIT     = 1 << 12;\r
+    public static final int CURSOR_VISIBLE_BIT     = 1 << 12;\r
     \r
     // public static int RESERVED1_BIT       = 1 << 1;\r
     // public static int RESERVED2_BIT       = 1 << 3;\r
@@ -109,7 +109,7 @@ public class SSSlideInfoAtom extends RecordAtom {
     // public static int RESERVED6_BIT       = 1 << 11;\r
     // public static int RESERVED7_BIT       = 1 << 13 | 1 << 14 | 1 << 15;\r
     \r
-    private static long _type = RecordTypes.SSSlideInfoAtom.typeID;\r
+    private static final long _type = RecordTypes.SSSlideInfoAtom.typeID;\r
 \r
     private byte[] _header;\r
 \r
index 953b0a25a1524c167fe0a031b2219133e8d9e261..5730021c9001b2fe45e8e1c4a0235dba45eb121d 100644 (file)
@@ -118,7 +118,7 @@ public final class StyleTextPropAtom extends RecordAtom
     }
 
     /** All the different kinds of paragraph properties we might handle */
-    public static TextProp[] paragraphTextPropTypes = new TextProp[] {
+    public static final TextProp[] paragraphTextPropTypes = new TextProp[] {
         new TextProp(0, 0x1, "hasBullet"),
         new TextProp(0, 0x2, "hasBulletFont"),
         new TextProp(0, 0x4, "hasBulletColor"),
@@ -141,7 +141,7 @@ public final class StyleTextPropAtom extends RecordAtom
         new TextProp(2, 0x200000, "textDirection")
     };
     /** All the different kinds of character properties we might handle */
-    public static TextProp[] characterTextPropTypes = new TextProp[] {
+    public static final TextProp[] characterTextPropTypes = new TextProp[] {
         new TextProp(0, 0x1, "bold"),
         new TextProp(0, 0x2, "italic"),
         new TextProp(0, 0x4, "underline"),
index 635e0ea265b6017daefd99e9aa4e6d1432d0369f..82e3c2060a888b63e4febcb0db00a91974177981 100644 (file)
 
 package org.apache.poi.hslf.usermodel;
 
-import org.apache.poi.util.LittleEndian;
-import org.apache.poi.util.POILogger;
-import org.apache.poi.util.POILogFactory;
-import org.apache.poi.hslf.model.Picture;
-import org.apache.poi.hslf.blip.*;
-import org.apache.poi.hslf.exceptions.HSLFException;
-
-import java.io.OutputStream;
+import java.awt.Graphics2D;
 import java.io.IOException;
+import java.io.OutputStream;
 import java.security.MessageDigest;
 import java.security.NoSuchAlgorithmException;
-import java.awt.*;
+
+import org.apache.poi.hslf.blip.BitmapPainter;
+import org.apache.poi.hslf.blip.DIB;
+import org.apache.poi.hslf.blip.EMF;
+import org.apache.poi.hslf.blip.ImagePainter;
+import org.apache.poi.hslf.blip.JPEG;
+import org.apache.poi.hslf.blip.PICT;
+import org.apache.poi.hslf.blip.PNG;
+import org.apache.poi.hslf.blip.WMF;
+import org.apache.poi.hslf.exceptions.HSLFException;
+import org.apache.poi.hslf.model.Picture;
+import org.apache.poi.util.LittleEndian;
+import org.apache.poi.util.POILogFactory;
+import org.apache.poi.util.POILogger;
 
 /**
  * A class that represents image data contained in a slide show.
@@ -77,7 +84,7 @@ public abstract class PictureData {
      */
     protected abstract int getSignature();
 
-    protected static ImagePainter[] painters = new ImagePainter[8];
+    protected static final ImagePainter[] painters = new ImagePainter[8];
     static {
         PictureData.setImagePainter(Picture.PNG, new BitmapPainter());
         PictureData.setImagePainter(Picture.JPEG, new BitmapPainter());
index 78a536e27c4bf251f6c2d52221883f6e4ab7f52a..74b77bf54e2a83944807f90e36da393324d3c886 100644 (file)
@@ -40,7 +40,7 @@ public abstract class PropertyNode<T extends PropertyNode<T>>  implements Compar
     public static final class EndComparator implements
             Comparator<PropertyNode<?>>
     {
-        public static EndComparator instance = new EndComparator();
+        public static final EndComparator instance = new EndComparator();
 
         public int compare( PropertyNode<?> o1, PropertyNode<?> o2 )
         {
@@ -54,7 +54,7 @@ public abstract class PropertyNode<T extends PropertyNode<T>>  implements Compar
     public static final class StartComparator implements
             Comparator<PropertyNode<?>>
     {
-        public static StartComparator instance = new StartComparator();
+        public static final StartComparator instance = new StartComparator();
 
         public int compare( PropertyNode<?> o1, PropertyNode<?> o2 )
         {