Make value copyable (after all)

2 files changed, 7 insertions, 10 deletions

diff --git a/include/crtp_tag.h b/include/crtp_tag.h
index 928759d6b8..a871866a2c 100644
--- a/include/crtp_tag.h
+++ b/include/crtp_tag.h
@@ -55,13 +55,13 @@ namespace detail
     }
 
     //TODO: Add copy constructors for tags that are missing it before this becomes useable
-    /*template<class Sub>
+    template<class Sub>
     std::unique_ptr<tag> crtp_tag<Sub>::clone() const&
     {
         return std::unique_ptr<tag>(
             new Sub(static_cast<const Sub&>(*this))
         );
-    }*/
+    }
 
     template<class Sub>
     std::unique_ptr<tag> crtp_tag<Sub>::move_clone() &&
diff --git a/include/value.h b/include/value.h
index 64e79efe9c..02937c8f5e 100644
--- a/include/value.h
+++ b/include/value.h
@@ -39,12 +39,14 @@ public:
     explicit value(std::unique_ptr<tag>&& t);
     explicit value(tag&& t);
 
-    //Movable but not (implicitly) copyable
-    value(const value&) = delete;
+    //Moving
     value(value&&) = default;
-    value& operator=(const value&) = delete;
     value& operator=(value&&) = default;
 
+    //Copying
+    explicit value(const value& rhs);
+    value& operator=(const value& rhs);
+
     /**
      * @brief Assigns the given value to the tag if the type matches
      * @throw std::bad_cast if the type of @c t is not the same as the type
@@ -128,11 +130,6 @@ public:
     value& operator[](const std::string& key);
     value& operator[](const char* key); //need this overload because of conflict with built-in operator[]
 
-    /**
-     * @brief Creates a copy of the value
-     */
-    value copy() const;
-
     std::unique_ptr<tag>& get_ptr();
     const std::unique_ptr<tag>& get_ptr() const;
     void set_ptr(std::unique_ptr<tag>&& t);