Merge pull request #20 from niborg/fix_track_append

Fix duplication of points on appending segment to track

lib/gpx/track.rb

@@ -55,7 +55,6 @@ def initialize(opts = {})
     def append_segment(seg)
       update_meta_data(seg)
       @segments << seg
-      @points.concat(seg.points) unless seg.nil?
     end
 
     # Returns true if the given time occurs within any of the segments of this track.
@@ -125,7 +124,7 @@ def recalculate_distance
         @distance += seg.distance
       end
     end
- 
+
     protected
 
     def update_meta_data(seg)

tests/track_test.rb

@@ -69,4 +69,13 @@ def test_track_delete
       assert_equal(-109.447045, @track.bounds.max_lon)
    end
 
+   def test_append_segment
+     trk = GPX::Track.new
+     seg = GPX::Segment.new(track: trk)
+     pt = GPX::TrackPoint.new(lat: -118, lon: 34)
+     seg.append_point(pt)
+     trk.append_segment(seg)
+     assert_equal(1, trk.points.size)
+   end
+
 end