[PATCH 1 of 2] Fixed some warnings

Wald Commits scm-commit at wald.intevation.org
Tue Feb 12 14:08:23 CET 2019


# HG changeset patch
# User gernotbelger
# Date 1549976787 -3600
# Node ID 8f59aa01c22af2d1d8cdc54b65c8653aa12062ed
# Parent  bf79a865f6dfed148149499d6b737cbdd15f2b72
Fixed some warnings

diff -r bf79a865f6df -r 8f59aa01c22a artifacts/src/main/java/org/dive4elements/river/exports/XYChartGenerator.java
--- a/artifacts/src/main/java/org/dive4elements/river/exports/XYChartGenerator.java	Tue Feb 12 12:03:49 2019 +0100
+++ b/artifacts/src/main/java/org/dive4elements/river/exports/XYChartGenerator.java	Tue Feb 12 14:06:27 2019 +0100
@@ -19,15 +19,24 @@
 import javax.swing.ImageIcon;
 
 import org.apache.log4j.Logger;
+import org.dive4elements.artifactdatabase.state.ArtifactAndFacet;
+import org.dive4elements.artifacts.CallContext;
+import org.dive4elements.river.jfree.AxisDataset;
+import org.dive4elements.river.jfree.Bounds;
+import org.dive4elements.river.jfree.CollisionFreeXYTextAnnotation;
+import org.dive4elements.river.jfree.DoubleBounds;
+import org.dive4elements.river.jfree.RiverAnnotation;
+import org.dive4elements.river.jfree.StyledAreaSeriesCollection;
+import org.dive4elements.river.jfree.StyledXYSeries;
+import org.dive4elements.river.themes.ThemeDocument;
 import org.jfree.chart.ChartFactory;
 import org.jfree.chart.JFreeChart;
-import org.jfree.chart.LegendItem;
 import org.jfree.chart.annotations.XYAnnotation;
 import org.jfree.chart.annotations.XYImageAnnotation;
 import org.jfree.chart.annotations.XYTextAnnotation;
+import org.jfree.chart.axis.LogarithmicAxis;
 import org.jfree.chart.axis.NumberAxis;
 import org.jfree.chart.axis.ValueAxis;
-import org.jfree.chart.axis.LogarithmicAxis;
 import org.jfree.chart.plot.Marker;
 import org.jfree.chart.plot.PlotOrientation;
 import org.jfree.chart.plot.XYPlot;
@@ -36,21 +45,9 @@
 import org.jfree.data.xy.XYDataset;
 import org.jfree.data.xy.XYSeries;
 import org.jfree.data.xy.XYSeriesCollection;
-import org.jfree.ui.RectangleInsets;
 import org.json.JSONArray;
 import org.json.JSONException;
 
-import org.dive4elements.artifactdatabase.state.ArtifactAndFacet;
-import org.dive4elements.artifacts.CallContext;
-import org.dive4elements.river.jfree.Bounds;
-import org.dive4elements.river.jfree.CollisionFreeXYTextAnnotation;
-import org.dive4elements.river.jfree.DoubleBounds;
-import org.dive4elements.river.jfree.RiverAnnotation;
-import org.dive4elements.river.jfree.StyledAreaSeriesCollection;
-import org.dive4elements.river.jfree.StyledXYSeries;
-import org.dive4elements.river.jfree.AxisDataset;
-import org.dive4elements.river.themes.ThemeDocument;
-
 
 /**
  * An abstract base class for creating XY charts.
@@ -385,7 +382,7 @@
             return;
         }
 
-        AxisDataset axisDataset = (AxisDataset) getAxisDataset(index);
+        AxisDataset axisDataset = getAxisDataset(index);
 
         if (visible) {
             axisDataset.addDataset(area);
@@ -484,11 +481,8 @@
 
         DoubleBounds dBounds = (DoubleBounds) bounds;
 
-        if (dBounds == null
-            || Double.isNaN((Double) dBounds.getLower())
-            || Double.isNaN((Double) dBounds.getUpper())) {
+        if ( Double.isNaN((Double) dBounds.getLower()) || Double.isNaN((Double) dBounds.getUpper()))
             return;
-        }
 
         Bounds old = getXBounds(index);
 
@@ -509,11 +503,8 @@
 
         DoubleBounds dBounds = (DoubleBounds) bounds;
 
-        if (dBounds == null
-            || Double.isNaN((Double) dBounds.getLower())
-            || Double.isNaN((Double) dBounds.getUpper())) {
+        if ( Double.isNaN((Double) dBounds.getLower()) || Double.isNaN((Double) dBounds.getUpper()))
             return;
-        }
 
         Bounds old = getYBounds(index);
 
@@ -631,7 +622,7 @@
                 return new Range(from, to);
             }
             catch (NumberFormatException nfe) {
-                log.warn("Wrong values for domain axis range.");
+                log.warn("Wrong values for domain axis range.", nfe);
             }
         }
 
@@ -662,7 +653,7 @@
                        : new Range(from, to);
             }
             catch (NumberFormatException nfe) {
-                log.warn("Wrong values for value axis range.");
+                log.warn("Wrong values for value axis range.", nfe);
             }
         }
 
@@ -910,7 +901,7 @@
         XYSeries series = new StyledXYSeries(aandf.getFacetName(), seriesName, theme);
 
         // Add text annotations for single points.
-        List<XYTextAnnotation> xy = new ArrayList<XYTextAnnotation>();
+        List<XYTextAnnotation> xy = new ArrayList<>();
 
         try {
             JSONArray points = new JSONArray((String) o);
@@ -929,7 +920,7 @@
             }
         }
         catch(JSONException e){
-            log.error("Could not decode json.");
+            log.error("Could not decode json.", e);
         }
 
         RiverAnnotation annotations =


More information about the Dive4Elements-commits mailing list