]> source.dussan.org Git - poi.git/commitdiff
Code cleanup via IntelliJ: Remove unnecessary semicolon
authorDominik Stadler <centic@apache.org>
Wed, 6 Apr 2016 19:49:47 +0000 (19:49 +0000)
committerDominik Stadler <centic@apache.org>
Wed, 6 Apr 2016 19:49:47 +0000 (19:49 +0000)
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1738031 13f79535-47bb-0310-9956-ffa450edef68

41 files changed:
src/examples/src/org/apache/poi/xssf/streaming/examples/HybridStreaming.java
src/java/org/apache/poi/ddf/EscherPropertyMetaData.java
src/java/org/apache/poi/poifs/crypt/CipherAlgorithm.java
src/java/org/apache/poi/poifs/crypt/HashAlgorithm.java
src/java/org/apache/poi/poifs/filesystem/Ole10Native.java
src/java/org/apache/poi/sl/draw/SLGraphics.java
src/java/org/apache/poi/sl/usermodel/TableCell.java
src/java/org/apache/poi/sl/usermodel/TextParagraph.java
src/java/org/apache/poi/sl/usermodel/TextShape.java
src/java/org/apache/poi/ss/format/CellNumberFormatter.java
src/java/org/apache/poi/ss/formula/functions/LinearRegressionFunction.java
src/java/org/apache/poi/ss/usermodel/FillPatternType.java
src/java/org/apache/poi/ss/usermodel/IgnoredErrorType.java
src/java/org/apache/poi/ss/usermodel/PaperSize.java
src/java/org/apache/poi/ss/util/CellReference.java
src/ooxml/java/org/apache/poi/openxml4j/opc/PackagePart.java
src/ooxml/java/org/apache/poi/util/OOXMLLite.java
src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFShape.java
src/ooxml/java/org/apache/poi/xssf/extractor/XSSFExportToXml.java
src/ooxml/java/org/apache/poi/xssf/model/ThemesTable.java
src/ooxml/java/org/apache/poi/xssf/usermodel/TextDirection.java
src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFDocument.java
src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFRun.java
src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFTable.java
src/ooxml/java/org/apache/poi/xwpf/usermodel/XWPFTableCell.java
src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFSheet.java
src/scratchpad/src/org/apache/poi/hslf/blip/JPEG.java
src/scratchpad/src/org/apache/poi/hslf/model/PPGraphics2D.java
src/scratchpad/src/org/apache/poi/hslf/model/textproperties/BitMaskTextProp.java
src/scratchpad/src/org/apache/poi/hslf/model/textproperties/TextPropCollection.java
src/scratchpad/src/org/apache/poi/hslf/record/PPDrawing.java
src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFSlideShow.java
src/scratchpad/src/org/apache/poi/hwmf/record/HwmfText.java
src/scratchpad/src/org/apache/poi/hwpf/usermodel/OfficeDrawing.java
src/scratchpad/testcases/org/apache/poi/hslf/model/TestSheet.java
src/scratchpad/testcases/org/apache/poi/hslf/model/TestSlideMaster.java
src/scratchpad/testcases/org/apache/poi/hslf/model/TestSlides.java
src/scratchpad/testcases/org/apache/poi/hslf/usermodel/TestAddingSlides.java
src/testcases/org/apache/poi/hssf/usermodel/TestBugs.java
src/testcases/org/apache/poi/hssf/usermodel/TestCellStyle.java
src/testcases/org/apache/poi/ss/formula/TestFunctionRegistry.java

index f2ceb5e1d4d9ce1f9f4a17cce74cdd7934bb589c..1e5515859dc6abfe2e571580356a836fa6dd86d2 100644 (file)
@@ -46,7 +46,7 @@ public class HybridStreaming {
                 if (!SHEET_TO_STREAM.equals(ctSheet.getName())) {
                     super.parseSheet(shIdMap, ctSheet);
                 }
-            };
+            }
         };
         
         // Having avoided a DOM-based parse of the sheet, we can stream it instead.
index 0c3105f57dae19520ab2e79f089b1c247f4ff6c6..b38e52df3e29ed34e14756329c1e35604f8ae71c 100644 (file)
@@ -31,7 +31,7 @@ public class EscherPropertyMetaData
     public final static byte TYPE_RGB = (byte) 2;
     public final static byte TYPE_SHAPEPATH = (byte) 3;
     public final static byte TYPE_SIMPLE = (byte)4;
-    public final static byte TYPE_ARRAY = (byte)5;;
+    public final static byte TYPE_ARRAY = (byte)5;
 
     private String description;
     private byte type;
index 5b5b1e186d3d45910243d2188d4e4d7bbcdbba19..c5887354db505f677550cf2c687b0c4b458cace3 100644 (file)
@@ -36,8 +36,7 @@ public enum CipherAlgorithm {
     des3_112(null, "DESede", -1, 128, new int[]{128}, 8, 32, "3DES_112", true),\r
     // only for digital signatures\r
     rsa(null, "RSA", -1, 1024, new int[]{1024, 2048, 3072, 4096}, -1, -1, "", false);\r
-    ;\r
-    \r
+\r
     public final CipherProvider provider;\r
     public final String jceId;\r
     public final int ecmaId;\r
index 6a490b01480dd27b27bb63960a83073b59460000..24c43325d9e5eed72e029da1322466e9a2896996 100644 (file)
@@ -35,7 +35,6 @@ public enum HashAlgorithm {
     whirlpool("Whirlpool",     -1,  "WHIRLPOOL", 64, "HMac-Whirlpool", true),\r
     // only for xml signing\r
     sha224   (  "SHA-224",     -1,     "SHA224", 28,     "HmacSHA224", true);\r
-    ;\r
 \r
     public final String jceId;\r
     public final int ecmaId;\r
index 4aee1e07b2e828dc2ffddb095fe934950ba6f1b8..99b7ab39313c7ee3f2bdcf10519fb0b381585724 100644 (file)
@@ -14,9 +14,9 @@
    See the License for the specific language governing permissions and\r
    limitations under the License.\r
 ==================================================================== */\r
-
-package org.apache.poi.poifs.filesystem;
-
+\r
+package org.apache.poi.poifs.filesystem;\r
+\r
 import java.io.ByteArrayOutputStream;\r
 import java.io.IOException;\r
 import java.io.OutputStream;\r
@@ -30,24 +30,24 @@ import org.apache.poi.util.StringUtil;
  * Represents an Ole10Native record which is wrapped around certain binary\r
  * files being embedded in OLE2 documents.\r
  *\r
- * @author Rainer Schwarze
- */
-public class Ole10Native {
-
-    public static final String OLE10_NATIVE = "\u0001Ole10Native";
-    protected static final String ISO1 = "ISO-8859-1";
-  
+ * @author Rainer Schwarze\r
+ */\r
+public class Ole10Native {\r
+\r
+    public static final String OLE10_NATIVE = "\u0001Ole10Native";\r
+    protected static final String ISO1 = "ISO-8859-1";\r
+  \r
     // (the fields as they appear in the raw record:)\r
-    private int totalSize;             // 4 bytes, total size of record not including this field
-    private short flags1 = 2;          // 2 bytes, unknown, mostly [02 00]
-    private String label;              // ASCIIZ, stored in this field without the terminating zero
-    private String fileName;           // ASCIIZ, stored in this field without the terminating zero
-    private short flags2 = 0;          // 2 bytes, unknown, mostly [00 00]
-    private short unknown1 = 3;        // see below
-    private String command;            // ASCIIZ, stored in this field without the terminating zero
-    private byte[] dataBuffer;         // varying size, the actual native data
-    private short flags3 = 0;          // some final flags? or zero terminators?, sometimes not there
-  
+    private int totalSize;             // 4 bytes, total size of record not including this field\r
+    private short flags1 = 2;          // 2 bytes, unknown, mostly [02 00]\r
+    private String label;              // ASCIIZ, stored in this field without the terminating zero\r
+    private String fileName;           // ASCIIZ, stored in this field without the terminating zero\r
+    private short flags2 = 0;          // 2 bytes, unknown, mostly [00 00]\r
+    private short unknown1 = 3;        // see below\r
+    private String command;            // ASCIIZ, stored in this field without the terminating zero\r
+    private byte[] dataBuffer;         // varying size, the actual native data\r
+    private short flags3 = 0;          // some final flags? or zero terminators?, sometimes not there\r
+  \r
     /**\r
      * the field encoding mode - merely a try-and-error guess ...\r
      **/ \r
@@ -63,16 +63,16 @@ public class Ole10Native {
         /**\r
          * the data is stored raw after the length field and the flags1 field\r
          */\r
-        compact;\r
+        compact\r
     }\r
     \r
     private EncodingMode mode;\r
 \r
     \r
     \r
-    /**
-     * Creates an instance of this class from an embedded OLE Object. The OLE Object is expected
-     * to include a stream &quot;{01}Ole10Native&quot; which contains the actual
+    /**\r
+     * Creates an instance of this class from an embedded OLE Object. The OLE Object is expected\r
+     * to include a stream &quot;{01}Ole10Native&quot; which contains the actual\r
      * data relevant for this class.\r
      *\r
      * @param poifs POI Filesystem object\r
@@ -102,18 +102,18 @@ public class Ole10Native {
        assert(readBytes == data.length);\r
   \r
        return new Ole10Native(data, 0);\r
-    }
-    
-    /**
-     * Creates an instance and fills the fields based on ... the fields
-     */
-    public Ole10Native(String label, String filename, String command, byte[] data) {
-       setLabel(label);
-       setFileName(filename);
-       setCommand(command);
+    }\r
+    \r
+    /**\r
+     * Creates an instance and fills the fields based on ... the fields\r
+     */\r
+    public Ole10Native(String label, String filename, String command, byte[] data) {\r
+       setLabel(label);\r
+       setFileName(filename);\r
+       setCommand(command);\r
        setDataBuffer(data);\r
-       mode = EncodingMode.parsed;
-    }
+       mode = EncodingMode.parsed;\r
+    }\r
 \r
     /**\r
      * Creates an instance and fills the fields based on the data in the given buffer.\r
@@ -128,11 +128,11 @@ public class Ole10Native {
     public Ole10Native(byte[] data, int offset, boolean plain) throws Ole10NativeException {\r
         this(data, offset);\r
     }\r
-    
-    /**
-     * Creates an instance and fills the fields based on the data in the given buffer.
-     *
-     * @param data   The buffer containing the Ole10Native record
+    \r
+    /**\r
+     * Creates an instance and fills the fields based on the data in the given buffer.\r
+     *\r
+     * @param data   The buffer containing the Ole10Native record\r
      * @param offset The start offset of the record in the buffer\r
      * @throws Ole10NativeException on invalid or unexcepted data format\r
      */\r
@@ -174,25 +174,25 @@ public class Ole10Native {
             ofs += len;\r
     \r
             flags2 = LittleEndian.getShort(data, ofs);\r
-            ofs += LittleEndianConsts.SHORT_SIZE;
-            
-            unknown1 = LittleEndian.getShort(data, ofs);
-            ofs += LittleEndianConsts.SHORT_SIZE;
-          
-            len = LittleEndian.getInt(data, ofs);
+            ofs += LittleEndianConsts.SHORT_SIZE;\r
+            \r
+            unknown1 = LittleEndian.getShort(data, ofs);\r
+            ofs += LittleEndianConsts.SHORT_SIZE;\r
+          \r
+            len = LittleEndian.getInt(data, ofs);\r
             ofs += LittleEndianConsts.INT_SIZE;\r
-            command = StringUtil.getFromCompressedUnicode(data, ofs, len - 1);
-            ofs += len;
-            
-            if (totalSize < ofs) {
-                throw new Ole10NativeException("Invalid Ole10Native");
-            }
-          
-            dataSize = LittleEndian.getInt(data, ofs);
-            ofs += LittleEndianConsts.INT_SIZE;
-          
-            if (dataSize < 0 || totalSize - (ofs - LittleEndianConsts.INT_SIZE) < dataSize) {
-                throw new Ole10NativeException("Invalid Ole10Native");
+            command = StringUtil.getFromCompressedUnicode(data, ofs, len - 1);\r
+            ofs += len;\r
+            \r
+            if (totalSize < ofs) {\r
+                throw new Ole10NativeException("Invalid Ole10Native");\r
+            }\r
+          \r
+            dataSize = LittleEndian.getInt(data, ofs);\r
+            ofs += LittleEndianConsts.INT_SIZE;\r
+          \r
+            if (dataSize < 0 || totalSize - (ofs - LittleEndianConsts.INT_SIZE) < dataSize) {\r
+                throw new Ole10NativeException("Invalid Ole10Native");\r
             }\r
             break;\r
         }\r
@@ -205,11 +205,11 @@ public class Ole10Native {
         case unparsed:\r
             dataSize = totalSize;\r
             break;\r
-        }
-        
-        dataBuffer = new byte[dataSize];
-        System.arraycopy(data, ofs, dataBuffer, 0, dataSize);
-        ofs += dataSize;
+        }\r
+        \r
+        dataBuffer = new byte[dataSize];\r
+        System.arraycopy(data, ofs, dataBuffer, 0, dataSize);\r
+        ofs += dataSize;\r
     }\r
 \r
     /*\r
@@ -326,12 +326,12 @@ public class Ole10Native {
         return flags3;\r
     }\r
 \r
-    /**
-     * Have the contents printer out into an OutputStream, used when writing a
-     * file back out to disk (Normally, atom classes will keep their bytes
-     * around, but non atom classes will just request the bytes from their
-     * children, then chuck on their header and return)
-     */
+    /**\r
+     * Have the contents printer out into an OutputStream, used when writing a\r
+     * file back out to disk (Normally, atom classes will keep their bytes\r
+     * around, but non atom classes will just request the bytes from their\r
+     * children, then chuck on their header and return)\r
+     */\r
     public void writeOut(OutputStream out) throws IOException {\r
         // byte intbuf[] = new byte[LittleEndianConsts.INT_SIZE];\r
         // byte shortbuf[] = new byte[LittleEndianConsts.SHORT_SIZE];\r
@@ -377,7 +377,7 @@ public class Ole10Native {
         }\r
 \r
     }\r
-
+\r
     public void setFlags1(short flags1) {\r
         this.flags1 = flags1;\r
     }\r
@@ -409,4 +409,4 @@ public class Ole10Native {
     public void setDataBuffer(byte dataBuffer[]) {\r
         this.dataBuffer = dataBuffer;\r
     }\r
-}
+}\r
index b4bc14ae965d6a3b2d59a05ba0912844d66a16b9..1deca63d1396dc28f98cee6d2fac842b4c51c29b 100644 (file)
@@ -918,7 +918,6 @@ public final class SLGraphics extends Graphics2D implements Cloneable {
      * @see         java.awt.Graphics#create\r
      */\r
     public void dispose() {\r
-        ;\r
     }\r
 \r
     /**\r
@@ -1145,7 +1144,6 @@ public final class SLGraphics extends Graphics2D implements Cloneable {
     }\r
 \r
     public void copyArea(int x, int y, int width, int height, int dx, int dy) {\r
-        ;\r
     }\r
 \r
     /**\r
index 5ad4c4c296ee83c9a712076f9678f7249eeb78a7..0b8d7dc549e0c0cf9450a129f5d47ea48d221185 100644 (file)
@@ -26,8 +26,8 @@ public interface TableCell<
     S extends Shape<S,P>,\r
     P extends TextParagraph<S,P,?>\r
 > extends TextShape<S,P> {\r
-    enum BorderEdge { bottom, left, top, right };\r
-    \r
+    enum BorderEdge { bottom, left, top, right }\r
+\r
     /**\r
      * Return line style of given edge or {@code null} if border is not defined\r
      *\r
index 61c06a9e4c8d3c28e4e7b4531392260101bc7147..50e75becbb78daa1b163a2f18abe8ca495efbe2d 100644 (file)
@@ -104,7 +104,7 @@ public interface TextParagraph<
          * This is different than BASELINE because of letters such as "g", "q", and "y".\r
          * Also known as "UpholdFixed"\r
          */\r
-        BOTTOM\r
+        BOTTOM\r
     }\r
     \r
     public interface BulletStyle {\r
index d2f66e52f1d0036bd7c9357d96b400ebd419418f..a4a61824d30faa68faf652b774a50e663513d657 100644 (file)
@@ -50,7 +50,7 @@ public interface TextShape<
          * Determines if all of the text is vertical\r
          * ("one letter on top of another").\r
          */\r
-        STACKED;\r
+        STACKED\r
     }\r
 \r
     /**\r
index 230d245546d28274865be2f1c7b707f998c34f22..668149f85a0a10536eb586c9c073f9d129a2af88 100644 (file)
@@ -96,7 +96,7 @@ public class CellNumberFormatter extends CellFormatter {
         public void simpleValue(StringBuffer toAppendTo, Object value) {
             formatValue(toAppendTo, value);
         }
-    };
+    }
 
 
     /**
index 870d08bc7576414c5713019b4c7f8ee7c1c76520..73a0204be05567cfcd7e989eabc9e78e5663fcd5 100644 (file)
@@ -106,7 +106,7 @@ public final class LinearRegressionFunction extends Fixed2ArgFunction {
                }
        }
 
-       public enum FUNCTION {INTERCEPT, SLOPE};
+       public enum FUNCTION {INTERCEPT, SLOPE}
        public FUNCTION function;
        
        public LinearRegressionFunction(FUNCTION function) {
index 25c8495b0e377dd4aa5e98e46387041535bb4cae..e34ed34b44a6357e8f5c0a292acf9df27105f366 100644 (file)
@@ -78,6 +78,6 @@ public enum FillPatternType {
      LESS_DOTS,
 
     /**  Least Dots */
-     LEAST_DOTS;
+     LEAST_DOTS
 
 }
index 9939ec1cd7ec121c131b76a9be79e7d7798bcf65..2843de4de32e2e18e6743ee328f06fca3747e7d7 100644 (file)
@@ -79,5 +79,5 @@ public enum IgnoredErrorType {
      * Whether to check for unprotected formulas.
      * HSSF + XSSF.
      */
-    UNLOCKED_FORMULA;
+    UNLOCKED_FORMULA
 }
index cebab20381c0e5ebd4e6241f380c80d4482806c9..abd65f3eaa3576f39cbf2a37040ceac8f0e963f8 100644 (file)
@@ -39,5 +39,5 @@ public enum PaperSize {
     FOLIO_PAPER,
     QUARTO_PAPER,
     STANDARD_PAPER_10_14,
-    STANDARD_PAPER_11_17;
+    STANDARD_PAPER_11_17
 }
index cf0fa57dfd20be55650ad788e6aece489f95fe58..6eb8035aa24aa1c8a36cff9f820f30f89eafb833 100644 (file)
@@ -47,7 +47,7 @@ public class CellReference {
         NAMED_RANGE,
         COLUMN,
         ROW,
-        BAD_CELL_OR_NAMED_RANGE;
+        BAD_CELL_OR_NAMED_RANGE
     }
 
     /** The character ($) that signifies a row or column value is absolute instead of relative */
index 8f961c9335a6621a07b1877faf0a950c79970080..bcf457e1aee87742373669676049d1e90c038a85 100644 (file)
@@ -453,8 +453,7 @@ public abstract class PackagePart implements RelationshipSource, Comparable<Pack
                     return true;
             }
         } catch (InvalidFormatException e){
-            ;
-        }
+               }
         return false;
        }
 
index de19cb0ed97f3a407741a24365630f559562ce72..43a07066a71081dd857fe57049fefa4dcd5d7902 100644 (file)
@@ -37,12 +37,12 @@ import java.util.Map;
 import java.util.Vector;
 import java.util.jar.JarEntry;
 import java.util.jar.JarFile;
-import java.util.regex.Pattern;\r
+import java.util.regex.Pattern;
 
-import junit.framework.TestCase;\r
+import junit.framework.TestCase;
 
-import org.junit.Test;\r
-import org.junit.internal.TextListener;\r
+import org.junit.Test;
+import org.junit.internal.TextListener;
 import org.junit.runner.JUnitCore;import org.junit.runner.Result;
 
 /**
@@ -201,7 +201,7 @@ public final class OOXMLLite {
             if (TestCase.class.isAssignableFrom(testclass)
                 || checkForTestAnnotation(testclass)) {
                 out.add(testclass);
-            };
+            }
         }
     }
 
index 29ca2e6541a8b41cc7b82e133547e3fb65dd31c3..5db266e42a682f372aadbded660f5c5ca0e595c9 100644 (file)
@@ -182,7 +182,7 @@ public abstract class XSLFShape implements Shape<XSLFShape,XSLFTextParagraph> {
                     if (paint != null) {\r
                         setValue(paint);\r
                         return true;\r
-                    };\r
+                    }\r
                 }\r
                 \r
                 return false;\r
index 734a9f45464d85bd9372a5594cbd09c31b90ae91..d187b66ca188f4596f2f0df18c0cd48bb3c77f20 100644 (file)
@@ -303,7 +303,7 @@ public class XSSFExportToXml implements Comparator<String>{
               }
             break;
 
-        default: ;
+        default:
 
         }
         if (node instanceof Element) {
index 19a08b3aae4da49ab9de1544774eae2d38d58bdc..0fe4abf4269bf59f642b5e7ebfe26a67ade7e530 100644 (file)
@@ -59,7 +59,7 @@ public class ThemesTable extends POIXMLDocumentPart {
        }
        public final int idx;
        public final String name;
-   };
+   }
 
     private ThemeDocument theme;
 
index 111ab652fca29d279ce54cfc8c8ff56b02746ae9..f91fcca7a8313e7c2b154e6b00df001f8f5bdc23 100644 (file)
@@ -44,5 +44,5 @@ public enum TextDirection {
      * Determines if all of the text is vertical
      * ("one letter on top of another").
      */
-    STACKED;
+    STACKED
 }
index e5dd3056c6049ac2e4a814abe8d551c9c8dc73af..74de72db3361edf45afc8a232dd954f30dafaa70 100644 (file)
@@ -1227,7 +1227,7 @@ public class XWPFDocument extends POIXMLDocument implements Document, IBody {
      * @param zoomPercent A percent value denoting the zoom setting for this document.
      */
     public void setZoomPercent(long zoomPercent) {
-        settings.setZoomPercent(zoomPercent);;
+        settings.setZoomPercent(zoomPercent);
     }
     
     /**
index 3bbcac704f70fc80486ab84b1a5d94408b1107ad..ea931e734952c49a09844b9e2e878e3df08bdb46 100644 (file)
@@ -150,8 +150,6 @@ public class XWPFRun implements ISDTContents, IRunElement, CharacterRun {
         }
     }
 
-    ;
-
     /**
      * @deprecated Use {@link XWPFRun#XWPFRun(CTR, IRunBody)}
      */
index 7c9bb397f988b0c6ac3f08506c3c982e32b41030..d512521388ce15cb8bde2d3039ad092e5a171d4f 100644 (file)
@@ -74,7 +74,6 @@ public class XWPFTable implements IBodyElement, ISDTContents {
     protected StringBuffer text = new StringBuffer();
     protected List<XWPFTableRow> tableRows;
 
-    ;
     protected List<String> styleIDs;
     protected IBody part;
     private CTTbl ctTbl;
index d074e79e0192009c1be259a82ae9975dc02e3e75..39a92f41cfc21603be91358dbee65732af529a5b 100644 (file)
@@ -68,7 +68,6 @@ public class XWPFTableCell implements IBody, ICell {
     protected List<XWPFTable> tables = null;\r
     protected List<IBodyElement> bodyElements = null;\r
 \r
-    ;\r
     protected IBody part;\r
     private XWPFTableRow tableRow = null;\r
 \r
index 2b7616a29c357f6156e6a4f0e3f71fa84d580f0e..844e591bc815efc4011fe796a18ac45c3d70c42b 100644 (file)
@@ -1319,7 +1319,7 @@ public final class TestXSSFSheet extends BaseTestSheet {
 
     @Test(timeout=180000)
     public void bug51585() throws IOException {
-        XSSFTestDataSamples.openSampleWorkbook("51585.xlsx").close();;
+        XSSFTestDataSamples.openSampleWorkbook("51585.xlsx").close();
     }
 
     private XSSFWorkbook setupSheet(){
index c7b94a68451dc287bf7b9cd154258c52cdfbce4c..72cac5f90dc257f189726e533107521a3aafa9e6 100644 (file)
@@ -23,8 +23,8 @@ package org.apache.poi.hslf.blip;
  */
 public final class JPEG extends Bitmap {
 
-    public enum ColorSpace { rgb, cymk };
-    
+    public enum ColorSpace { rgb, cymk }
+
     private ColorSpace colorSpace = ColorSpace.rgb;
     
     @Override
index 7d6b0c262619b4e95426b71fc93e2d255fcf29ff..70229b1bd8e212acd67bac0f7e554ae1a4fe23ec 100644 (file)
@@ -928,7 +928,6 @@ public final class PPGraphics2D extends Graphics2D implements Cloneable {
      * @see         java.awt.Graphics#create
      */
     public void dispose() {
-        ;
     }
 
     /**
@@ -1155,7 +1154,6 @@ public final class PPGraphics2D extends Graphics2D implements Cloneable {
     }
 
     public void copyArea(int x, int y, int width, int height, int dx, int dy) {
-        ;
     }
 
     /**
index 2b626e643d3bbf998dde2fa09546a4cc575d5986..067c0af1600a4f073beaf4d238185d5e0e55d21c 100644 (file)
@@ -91,8 +91,8 @@ public abstract class BitMaskTextProp extends TextProp implements Cloneable {
         */
        @Override
        public int getValue() {
-           int val = dataValue, i = 0;;
-           for (int mask : subPropMasks) {
+           int val = dataValue, i = 0;
+               for (int mask : subPropMasks) {
                if (!subPropMatches[i++]) {
                    val &= ~mask;
                }
index b6ffeec9bd7093af27af026e29a87318199dfbff..e82b966e60e41b2598faddc6a723d1f4767506c4 100644 (file)
@@ -86,7 +86,7 @@ public class TextPropCollection {
     };
 
     public enum TextPropType {
-        paragraph, character;
+        paragraph, character
     }
     
     private int charactersCovered;
index 90b24ed9f2142feabfaa8e7d94211c734dfbfed5..ba36989bc0bfc1ab895833d1472cbae70624d14e 100644 (file)
@@ -159,7 +159,7 @@ public final class PPDrawing extends RecordAtom {
                final Record r1 = progBinaryTag.getChildRecords()[1];
                
                if (!(r0 instanceof CString)) { return null; }
-               if (!("___PPT9".equals(((CString) r0).getText()))) { return null; };
+               if (!("___PPT9".equals(((CString) r0).getText()))) { return null; }
                if (!(r1 instanceof BinaryTagDataBlob )) { return null; }
                final BinaryTagDataBlob blob = (BinaryTagDataBlob) r1;
                if (1 != blob.getChildRecords().length) { return null; }
index 78abb0ab0f5d343583767a3bc2417a5b3fe716ad..4269861e679761ad69e53e1f5b1fe5a0c00f0317 100644 (file)
@@ -95,8 +95,8 @@ import org.apache.poi.util.Units;
  */
 public final class HSLFSlideShow implements SlideShow<HSLFShape,HSLFTextParagraph>, Closeable {
     enum LoadSavePhase {
-        INIT, LOADED;
-    }
+        INIT, LOADED
+       }
     private static ThreadLocal<LoadSavePhase> loadSavePhase = new ThreadLocal<LoadSavePhase>();
     
     // What we're based on
index e4e4ecb4d8df7f22f2a8ef05992adf4fb89216d7..1649d13eee61de66381cf84e54a301b2b3257535 100644 (file)
@@ -335,13 +335,13 @@ public class HwmfText {
     public enum HwmfTextAlignment {\r
         LEFT,\r
         RIGHT,\r
-        CENTER;\r
+        CENTER\r
     }\r
     \r
     public enum HwmfTextVerticalAlignment {\r
         TOP,\r
         BOTTOM,\r
-        BASELINE;\r
+        BASELINE\r
     }\r
     \r
     /**\r
index 5165c9eb58c5a10ad9f64e032290ad3fff4b609b..4bed04e495f410f04dd172aaf5a26d600cefcfdf 100644 (file)
@@ -69,11 +69,11 @@ public interface OfficeDrawing
          * The shape is horizontally positioned at the right side of the page
          * element.
          */
-        RIGHT;
+        RIGHT
     }
 
     public enum HorizontalRelativeElement {
-        CHAR, MARGIN, PAGE, TEXT;
+        CHAR, MARGIN, PAGE, TEXT
     }
 
     public enum VerticalPositioning {
@@ -109,11 +109,11 @@ public interface OfficeDrawing
         /**
          * The shape is vertically positioned at the top of the page element
          */
-        TOP;
+        TOP
     }
 
     public enum VerticalRelativeElement {
-        LINE, MARGIN, PAGE, TEXT;
+        LINE, MARGIN, PAGE, TEXT
     }
 
     /**
index a5e9084b9e88a2a64408f4a8bc59ddf42f2ca533..1b5aa8dcc5c67a589fb4a869bcd9b4cb29deead7 100644 (file)
@@ -49,7 +49,7 @@ public final class TestSheet {
                 HSLFSlideShow ppt = new HSLFSlideShow(_slTests.openResourceAsStream(file));
                 doSlideShow(ppt);
             } catch (EncryptedPowerPointFileException e){
-                //skip encrypted ppt
+                //skip encrypted ppt
             }
         }
     }
index cd46ef98ec4974e7ee8d2e8277abed7337adbece..23a3d156fa5d98ee8b69002b7c9c9788af2bafb4 100644 (file)
@@ -162,7 +162,6 @@ public final class TestSlideMaster {
             }
         }
 
-        ;
         for (List<HSLFTextParagraph> tparas : slide.get(1).getTextParagraphs()) {
             HSLFTextParagraph tpara = tparas.get(0);
             if (tpara.getRunType() == TextHeaderAtom.TITLE_TYPE){
index a84caefd68c8a09d4f1c8dc9ed081ebf50e7d102..5ced9945c845181adf2d82e3511c3077c5926700 100644 (file)
@@ -133,7 +133,7 @@ public final class TestSlides {
         s2 = ppt.getSlides().get(1);
         assertEquals(257, s2._getSheetNumber());
         assertEquals(4, s2._getSheetRefId());
-        s3 = ppt.getSlides().get(2);;
+        s3 = ppt.getSlides().get(2);
         assertEquals(3, ppt.getSlides().size());
         assertEquals(258, s3._getSheetNumber());
         assertEquals(5, s3._getSheetRefId());
index 800df623e3edc4deed50939e20c7dbbe66146543..b89a5f6e2d5d05aff41513ee1c9685bd39a58855 100644 (file)
@@ -238,15 +238,13 @@ public final class TestAddingSlides extends TestCase {
             ppt.removeSlide(-1);
             fail("expected exception");
         } catch (Exception e){
-            ;
-        }
+               }
 
         try {
             ppt.removeSlide(2);
             fail("expected exception");
         } catch (Exception e){
-            ;
-        }
+               }
 
         assertEquals(1, slide1.getSlideNumber());
 
index 56071f6d93c8dde1250c0d3fb95692388a390579..e55680538f689e2e39da8a614d23daadb4055d1d 100644 (file)
@@ -838,7 +838,7 @@ public final class TestBugs extends BaseTestBugzillaIssues {
     public void bug28774() throws Exception {
         HSSFWorkbook wb = openSample("28774.xls");
         assertTrue("no errors reading sample xls", true);
-        writeOutAndReadBack(wb).close();;
+        writeOutAndReadBack(wb).close();
         assertTrue("no errors writing sample xls", true);
         wb.close();
     }
index 428d2bf96797647b9fd4758c2e76f7fd468e8caa..cba7a6615edaa96f7997a0964fbc17b309c26b5a 100644 (file)
@@ -452,8 +452,8 @@ public final class TestCellStyle extends TestCase {
         public Throwable getException() {
             return exception;
         }
-    };
-    
+    }
+
     public void test56563() throws Throwable {
         CellFormatBugExample threadA = new CellFormatBugExample("56563a.xls");
         threadA.start();
index 87f07ce69dac283b1870e1d4657bb82fdcedfc64..b6d118e89b3c3cbfbd93c128aba2650831dbb8ff 100644 (file)
@@ -49,8 +49,7 @@ public class TestFunctionRegistry extends TestCase {
                        cv = fe.evaluate(cellA);
             fail("expectecd exception");
                } catch (NotImplementedException e) {
-                       ;
-               }
+        }
 
         FunctionEval.registerFunction("FISHER", new Function() {
             public ValueEval evaluate(ValueEval[] args, int srcRowIndex, int srcColumnIndex) {
@@ -67,7 +66,6 @@ public class TestFunctionRegistry extends TestCase {
             cv = fe.evaluate(cellB);
             fail("expectecd exception");
         } catch (NotImplementedException e) {
-            ;
         }
 
         AnalysisToolPak.registerFunction("CUBEMEMBERPROPERTY", new FreeRefFunction() {