Issue 3493: Let Skyline's copy constructor use generated copy constructor

Originally connected with issue 3490, now separate.

diff --git a/lily/include/skyline.hh b/lily/include/skyline.hh
index 9a5473ddd50e708edcee23b7b6468dfafbef03c6..6846474857d476227c797ce021ec24fa1def8e38 100644 (file)
--- a/lily/include/skyline.hh
+++ b/lily/include/skyline.hh
@@ -66,7 +66,6 @@ private:
 
 public:
   Skyline ();
-  Skyline (Skyline const &src);
   Skyline (Direction sky);
   Skyline (vector<Box> const &bldgs, Axis a, Direction sky);
   Skyline (vector<Drul_array<Offset> > const &bldgs, Axis a, Direction sky);

diff --git a/lily/skyline.cc b/lily/skyline.cc
index 5073e69e14b3eaccb805a712e8979be7d9b8856f..8654f0d973d26af771c106c44d441908dcb9fb27 100644 (file)
--- a/lily/skyline.cc
+++ b/lily/skyline.cc
@@ -457,18 +457,6 @@ Skyline::Skyline ()
   empty_skyline (&buildings_);
 }
 
-Skyline::Skyline (Skyline const &src)
-{
-  sky_ = src.sky_;
-
-  /* doesn't a list's copy constructor do this? -- jneem */
-  for (list<Building>::const_iterator i = src.buildings_.begin ();
-       i != src.buildings_.end (); i++)
-    {
-      buildings_.push_back (Building ((*i)));
-    }
-}
-
 Skyline::Skyline (Direction sky)
 {
   sky_ = sky;