Revert "DetachedSong: remove explicitly-defaulted copy/move constructors"

This reverts commit 67b7d464. Turns out I was wrong, and mentioning these does make a difference: the implicit move constructor is not defined in the presence of a user-declared destructor.

Revert "DetachedSong: remove explicitly-defaulted copy/move constructors"
1292af47 · Max Kellermann · 4d88a099 · 1292af47
Commit 1292af47 authored Feb 08, 2017 by Max Kellermann
Show whitespace changes
Inline Side-by-side

Showing with 5 additions and 0 deletions

DetachedSong.hxx src/DetachedSong.hxx +5 -0

No files found.
--- a/src/DetachedSong.hxx
+++ b/src/DetachedSong.hxx
@@ -100,6 +100,11 @@ public:

 	~DetachedSong();

+	/* these are declared because the user-defined destructor
+	   above prevents them from being generated implicitly */
+	explicit DetachedSong(const DetachedSong &) = default;
+	DetachedSong(DetachedSong &&) = default;
+
 	gcc_pure
 	const char *GetURI() const {
 		return uri.c_str();