]> source.dussan.org Git - poi.git/commitdiff
Fix some IntelliJ warnings in XDGF area
authorDominik Stadler <centic@apache.org>
Mon, 12 Feb 2018 15:34:11 +0000 (15:34 +0000)
committerDominik Stadler <centic@apache.org>
Mon, 12 Feb 2018 15:34:11 +0000 (15:34 +0000)
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1823988 13f79535-47bb-0310-9956-ffa450edef68

25 files changed:
src/ooxml/java/org/apache/poi/xdgf/usermodel/XDGFBaseContents.java
src/ooxml/java/org/apache/poi/xdgf/usermodel/XDGFCell.java
src/ooxml/java/org/apache/poi/xdgf/usermodel/XDGFMasterContents.java
src/ooxml/java/org/apache/poi/xdgf/usermodel/XDGFMasters.java
src/ooxml/java/org/apache/poi/xdgf/usermodel/XDGFPageContents.java
src/ooxml/java/org/apache/poi/xdgf/usermodel/XDGFPages.java
src/ooxml/java/org/apache/poi/xdgf/usermodel/XDGFSheet.java
src/ooxml/java/org/apache/poi/xdgf/usermodel/XmlVisioDocument.java
src/ooxml/java/org/apache/poi/xdgf/usermodel/section/XDGFSection.java
src/ooxml/java/org/apache/poi/xdgf/usermodel/section/geometry/ArcTo.java
src/ooxml/java/org/apache/poi/xdgf/usermodel/section/geometry/Ellipse.java
src/ooxml/java/org/apache/poi/xdgf/usermodel/section/geometry/EllipticalArcTo.java
src/ooxml/java/org/apache/poi/xdgf/usermodel/section/geometry/GeometryRowFactory.java
src/ooxml/java/org/apache/poi/xdgf/usermodel/section/geometry/InfiniteLine.java
src/ooxml/java/org/apache/poi/xdgf/usermodel/section/geometry/LineTo.java
src/ooxml/java/org/apache/poi/xdgf/usermodel/section/geometry/MoveTo.java
src/ooxml/java/org/apache/poi/xdgf/usermodel/section/geometry/NURBSTo.java
src/ooxml/java/org/apache/poi/xdgf/usermodel/section/geometry/PolyLineTo.java
src/ooxml/java/org/apache/poi/xdgf/usermodel/section/geometry/RelCubBezTo.java
src/ooxml/java/org/apache/poi/xdgf/usermodel/section/geometry/RelEllipticalArcTo.java
src/ooxml/java/org/apache/poi/xdgf/usermodel/section/geometry/RelLineTo.java
src/ooxml/java/org/apache/poi/xdgf/usermodel/section/geometry/RelMoveTo.java
src/ooxml/java/org/apache/poi/xdgf/usermodel/section/geometry/RelQuadBezTo.java
src/ooxml/java/org/apache/poi/xdgf/usermodel/section/geometry/SplineKnot.java
src/ooxml/java/org/apache/poi/xdgf/usermodel/section/geometry/SplineStart.java

index 37927d486dadd60dbbc07160349e4adf79d88e83..f195089f88ba2fd1af340c6cbbce2ad9d449161d 100644 (file)
@@ -27,7 +27,6 @@ import java.util.Map;
 
 import org.apache.poi.POIXMLException;
 import org.apache.poi.openxml4j.opc.PackagePart;
-import org.apache.poi.openxml4j.opc.PackageRelationship;
 import org.apache.poi.util.Internal;
 import org.apache.poi.xdgf.exceptions.XDGFException;
 import org.apache.poi.xdgf.usermodel.shape.ShapeRenderer;
@@ -112,7 +111,7 @@ public class XDGFBaseContents extends XDGFXMLDocumentPart {
     /**
      * Draws the contents of a page onto a Graphics2D object
      *
-     * @param graphics
+     * @param graphics The context to draw on.
      */
     public void draw(Graphics2D graphics) {
         visitShapes(new ShapeRenderer(graphics));
index f7f329184428704dedc74555c981475aab845cd6..5b4ea2b52c9cc859acef31caa815d1d22323f007 100644 (file)
@@ -102,7 +102,7 @@ public class XDGFCell {
     }
 
     /**
-     * @param cell
+     * @param cell The type of the cell
      * @return A value converted to inches
      */
     public static Double parseVLength(CellType cell) {
index 8fd51324ad4c4e1cb81e8ce13b1dcbb9498fe05c..192fd46bb259b2b746fc468a12fd7ba01cde89f8 100644 (file)
@@ -47,9 +47,7 @@ public class XDGFMasterContents extends XDGFBaseContents {
 
             try {
                 _pageContents = MasterContentsDocument.Factory.parse(getPackagePart().getInputStream()).getMasterContents();
-            } catch (XmlException e) {
-                throw new POIXMLException(e);
-            } catch (IOException e) {
+            } catch (XmlException | IOException e) {
                 throw new POIXMLException(e);
             }
 
index fdbd43d2aaafae76df04a3f8a6eeefc19422dd78..2e2c3de23dcefaec96585af69daf6601b7232ce8 100644 (file)
@@ -62,9 +62,7 @@ public class XDGFMasters extends XDGFXMLDocumentPart {
         try {
             try {
                 _mastersObject = MastersDocument.Factory.parse(getPackagePart().getInputStream()).getMasters();
-            } catch (XmlException e) {
-                throw new POIXMLException(e);
-            } catch (IOException e) {
+            } catch (XmlException | IOException e) {
                 throw new POIXMLException(e);
             }
 
index 3dfe3c9cc2dc389ca58142dd695ffbd201df20aa..1edff81b79b247fef00adde7877cf3c4e21c26b8 100644 (file)
@@ -46,9 +46,7 @@ public class XDGFPageContents extends XDGFBaseContents {
         try {
             try {
                 _pageContents = PageContentsDocument.Factory.parse(getPackagePart().getInputStream()).getPageContents();
-            } catch (XmlException e) {
-                throw new POIXMLException(e);
-            } catch (IOException e) {
+            } catch (XmlException | IOException e) {
                 throw new POIXMLException(e);
             }
 
index 65d2f50cc37edd5577288c628cd584d60f7b6d93..9ac26de653595009251abe56ae290ab4eb01dacb 100644 (file)
@@ -61,9 +61,7 @@ public class XDGFPages extends XDGFXMLDocumentPart {
         try {
             try {
                 _pagesObject = PagesDocument.Factory.parse(getPackagePart().getInputStream()).getPages();
-            } catch (XmlException e) {
-                throw new POIXMLException(e);
-            } catch (IOException e) {
+            } catch (XmlException | IOException e) {
                 throw new POIXMLException(e);
             }
 
index 1459cfd8b9158dc6e8929bf23863f98b204df12b..c04b1e55d252af5b027bc5e501ec06538e4e5fa9 100644 (file)
@@ -127,7 +127,7 @@ public abstract class XDGFSheet {
     }
 
     public Color getFontColor() {
-        Color fontColor = null;
+        Color fontColor;
 
         if (_character != null) {
             fontColor = _character.getFontColor();
@@ -143,7 +143,7 @@ public abstract class XDGFSheet {
     }
 
     public Double getFontSize() {
-        Double fontSize = null;
+        Double fontSize;
 
         if (_character != null) {
             fontSize = _character.getFontSize();
index 071389200b1ee314f2dfa7f0df85639b4d84fe3d..c3e211d8ffa52829cc5d832dd049748f77d5914b 100644 (file)
@@ -26,7 +26,6 @@ import java.util.List;
 import org.apache.poi.POIXMLDocument;
 import org.apache.poi.POIXMLDocumentPart;
 import org.apache.poi.POIXMLException;
-import org.apache.poi.openxml4j.exceptions.OpenXML4JException;
 import org.apache.poi.openxml4j.opc.OPCPackage;
 import org.apache.poi.openxml4j.opc.PackagePart;
 import org.apache.poi.openxml4j.opc.PackageRelationshipTypes;
@@ -67,9 +66,7 @@ public class XmlVisioDocument extends POIXMLDocument {
 
         try {
             document = VisioDocumentDocument1.Factory.parse(getPackagePart().getInputStream()).getVisioDocument();
-        } catch (XmlException e) {
-            throw new POIXMLException(e);
-        } catch (IOException e) {
+        } catch (XmlException | IOException e) {
             throw new POIXMLException(e);
         }
 
@@ -84,7 +81,7 @@ public class XmlVisioDocument extends POIXMLDocument {
     }
 
     @Override
-    protected void onDocumentRead() throws IOException {
+    protected void onDocumentRead() {
 
         // by the time this gets called, all other document parts should
         // have been loaded, so it's safe to build the document structure
@@ -112,7 +109,7 @@ public class XmlVisioDocument extends POIXMLDocument {
      * Not currently implemented
      */
     @Override
-    public List<PackagePart> getAllEmbedds() throws OpenXML4JException {
+    public List<PackagePart> getAllEmbedds() {
         return new ArrayList<>();
     }
 
index d26fc888cf20928ea51d7942507ea977ef363884..add8f5aa4c901b678f15a64431a60bfa753dde57 100644 (file)
@@ -55,9 +55,7 @@ public abstract class XDGFSection {
             _sectionTypes.put("Reviewer",              GenericSection.class, SectionType.class, XDGFSheet.class);
             _sectionTypes.put("Annotation",    GenericSection.class, SectionType.class, XDGFSheet.class);
             _sectionTypes.put("ActionTag",             GenericSection.class, SectionType.class, XDGFSheet.class);
-        } catch (NoSuchMethodException e) {
-            throw new POIXMLException("Internal error");
-        } catch (SecurityException e) {
+        } catch (NoSuchMethodException | SecurityException e) {
             throw new POIXMLException("Internal error");
         }
 
index be2a291c14940e0257dc8cdcbeaa412de13d141c..9b3908c5ce7080f0e8b80e4b34d2879e920957f7 100644 (file)
@@ -71,10 +71,7 @@ public class ArcTo implements GeometryRow {
         if (deleted != null)
             return deleted;
 
-        if (_master != null)
-            return _master.getDel();
-
-        return false;
+        return _master != null && _master.getDel();
     }
 
     public Double getX() {
index ba4d3ff67cc825f2513b1a868211276f0243cbeb..ab05c465c3b39de9b59617a3816507f72857c490 100644 (file)
@@ -82,10 +82,7 @@ public class Ellipse implements GeometryRow {
         if (deleted != null)
             return deleted;
 
-        if (_master != null)
-            return _master.getDel();
-
-        return false;
+        return _master != null && _master.getDel();
     }
 
     public Double getX() {
index 17db67c1fc1db6a5df63d36570d56048cefed49e..00f40769356fa10221f2f9ea001f38064c2d9737 100644 (file)
@@ -94,10 +94,7 @@ public class EllipticalArcTo implements GeometryRow {
         if (deleted != null)
             return deleted;
 
-        if (_master != null)
-            return _master.getDel();
-
-        return false;
+        return _master != null && _master.getDel();
     }
 
     public Double getX() {
index aa9c336d56117ed763b05734b7b696a41b5e3b86..f06280686b79a2d0e64e86ce8ae2e9bfc45bb4d8 100644 (file)
@@ -48,9 +48,7 @@ public class GeometryRowFactory {
             _rowTypes.put("RelQuadBezTo", RelQuadBezTo.class, RowType.class);
             _rowTypes.put("SplineKnot", SplineKnot.class, RowType.class);
             _rowTypes.put("SplineStart", SplineStart.class, RowType.class);
-        } catch (NoSuchMethodException e) {
-            throw new POIXMLException("Internal error", e);
-        } catch (SecurityException e) {
+        } catch (NoSuchMethodException | SecurityException e) {
             throw new POIXMLException("Internal error", e);
         }
 
index 0fc79c231eb14e2306e1ad2b0c6ccd2e8331753a..c70af455d6291368dfc2f5ff0312378556bf6b39 100644 (file)
@@ -80,10 +80,7 @@ public class InfiniteLine implements GeometryRow {
         if (deleted != null)
             return deleted;
 
-        if (_master != null)
-            return _master.getDel();
-
-        return false;
+        return _master != null && _master.getDel();
     }
 
     public Double getX() {
index 9c24988b386242b9de1f9766035b590ac749d6ce..14fa036db571b25c6f5d0485937b5c781ccc3b9c 100644 (file)
@@ -63,10 +63,7 @@ public class LineTo implements GeometryRow {
         if (deleted != null)
             return deleted;
 
-        if (_master != null)
-            return _master.getDel();
-
-        return false;
+        return _master != null && _master.getDel();
     }
 
     public Double getX() {
index 297b213e06d8060e9bc3f830ae32c4410c1cb1bb..28e3d89996c94e18d830a3ca4a1fad59805d4950 100644 (file)
@@ -68,10 +68,7 @@ public class MoveTo implements GeometryRow {
         if (deleted != null)
             return deleted;
 
-        if (_master != null)
-            return _master.getDel();
-
-        return false;
+        return _master != null && _master.getDel();
     }
 
     public Double getX() {
index f9458026518280e01b29662962c0c5f9becb3653..832d6fac9a14eb8b8d0560ba9cda1b48c1c409a9 100644 (file)
@@ -93,10 +93,7 @@ public class NURBSTo implements GeometryRow {
         if (deleted != null)
             return deleted;
 
-        if (_master != null)
-            return _master.getDel();
-
-        return false;
+        return _master != null && _master.getDel();
     }
 
     public Double getX() {
index 4b6d37ae5dbf436eb21a113706613cf964c6a7be..77e73d95a0cc60373e20d5e533df152cd0b9b595 100644 (file)
@@ -67,10 +67,7 @@ public class PolyLineTo implements GeometryRow {
         if (deleted != null)
             return deleted;
 
-        if (_master != null)
-            return _master.getDel();
-
-        return false;
+        return _master != null && _master.getDel();
     }
 
     public Double getX() {
index 3988de313d07b08cd663117bb609c6d58f206540..10c51bac5603a2a49cd6cd17c68e47d3f93b1080 100644 (file)
@@ -89,10 +89,7 @@ public class RelCubBezTo implements GeometryRow {
         if (deleted != null)
             return deleted;
 
-        if (_master != null)
-            return _master.getDel();
-
-        return false;
+        return _master != null && _master.getDel();
     }
 
     public Double getX() {
index d8fe5c69c316c5d204de9339254c18e75e4b708a..bc4b659a21b3b580c2fd90f70146fdb9686ac100 100644 (file)
@@ -86,10 +86,7 @@ public class RelEllipticalArcTo implements GeometryRow {
         if (deleted != null)
             return deleted;
 
-        if (_master != null)
-            return _master.getDel();
-
-        return false;
+        return _master != null && _master.getDel();
     }
 
     public Double getX() {
index dc552112955d65673882336555ac47c9d97f9cec..fb57e4c5e9dc6e1674e4b1b818969b1beccf0a67 100644 (file)
@@ -62,10 +62,7 @@ public class RelLineTo implements GeometryRow {
         if (deleted != null)
             return deleted;
 
-        if (_master != null)
-            return _master.getDel();
-
-        return false;
+        return _master != null && _master.getDel();
     }
 
     public Double getX() {
index 0a2f9bf9d6b3c0e59f134f69c90515be859e2de2..3e1652fc001c1a4fc9a5e09dfee6f713357b2f76 100644 (file)
@@ -58,10 +58,7 @@ public class RelMoveTo implements GeometryRow {
         if (deleted != null)
             return deleted;
 
-        if (_master != null)
-            return _master.getDel();
-
-        return false;
+        return _master != null && _master.getDel();
     }
 
     public Double getX() {
index ae070254840870264ecc4b3db0725be10ebf2b8f..50d3c7d354da17ef8b0761b60b2d1acb750bec2d 100644 (file)
@@ -81,10 +81,7 @@ public class RelQuadBezTo implements GeometryRow {
         if (deleted != null)
             return deleted;
 
-        if (_master != null)
-            return _master.getDel();
-
-        return false;
+        return _master != null && _master.getDel();
     }
 
     public Double getX() {
index 640b10ea48e8372be8f48a2ad0cd0281890afc4a..10765091cf501d86783775310ac2550d6a5c571a 100644 (file)
@@ -70,10 +70,7 @@ public class SplineKnot implements GeometryRow {
         if (deleted != null)
             return deleted;
 
-        if (_master != null)
-            return _master.getDel();
-
-        return false;
+        return _master != null && _master.getDel();
     }
 
     public Double getX() {
index a427de6f1fe74c576813acf398ade299b055c3fc..3aed3566daaadda7e2a834c5b9b6912dfc117ffc 100644 (file)
@@ -84,10 +84,7 @@ public class SplineStart implements GeometryRow {
         if (deleted != null)
             return deleted;
 
-        if (_master != null)
-            return _master.getDel();
-
-        return false;
+        return _master != null && _master.getDel();
     }
 
     public Double getX() {