# HG changeset patch
# User Sascha L. Teichmann <sascha.teichmann@intevation.de>
# Date 1326723412 0
# Node ID c5d83366d0b14c8bdb434f8b0a95488263248312
# Parent  d183ae164cfc866995121641e5920045e163e8db
CrossSectionLine: Simplified creation of Point2Ds.

flys-backend/trunk@3687 c6561f87-3c4e-4783-a992-168aeb5c3f6f

diff -r d183ae164cfc -r c5d83366d0b1 flys-backend/ChangeLog
--- a/flys-backend/ChangeLog	Tue Jan 10 10:33:14 2012 +0000
+++ b/flys-backend/ChangeLog	Mon Jan 16 14:16:52 2012 +0000
@@ -1,3 +1,8 @@
+2012-01-16	Sascha L. Teichmann	<sascha.teichmann@intevation.de>
+
+	* src/main/java/de/intevation/flys/model/CrossSectionLine.java
+	  (fetchCrossSectionLinesPoints): Simplified and prevent reallocations.
+
 2012-01-10	Sascha L. Teichmann	<sascha.teichmann@intevation.de>
 
 	* src/main/java/de/intevation/flys/importer/parsers/StaFileParser.java:
diff -r d183ae164cfc -r c5d83366d0b1 flys-backend/src/main/java/de/intevation/flys/model/CrossSectionLine.java
--- a/flys-backend/src/main/java/de/intevation/flys/model/CrossSectionLine.java	Tue Jan 10 10:33:14 2012 +0000
+++ b/flys-backend/src/main/java/de/intevation/flys/model/CrossSectionLine.java	Mon Jan 16 14:16:52 2012 +0000
@@ -119,18 +119,13 @@
 
 
     public List<Point2D> fetchCrossSectionLinesPoints() {
-        List<Point2D> points = new ArrayList<Point2D>();
-
-        List<CrossSectionPoint> linePoints = getPoints();
 
-        if (linePoints.isEmpty()) {
-            logger.info("No points in selected CrossSectionLine.");
-            return points;
-        }
+        List<CrossSectionPoint> linePoints = 
+            new ArrayList<CrossSectionPoint>(getPoints());
 
-        linePoints = new ArrayList(linePoints);
         Collections.sort(linePoints, COL_POS_CMP);
 
+        List<Point2D> points = new ArrayList<Point2D>(linePoints.size());
         for (CrossSectionPoint p: linePoints) {
             double x = p.getX().doubleValue();
             double y = p.getY().doubleValue();