Removed entirely unnecessary _copy_value API from web::json::details::_Value class.

Author: stgates

Release/include/cpprest/json.h

@@ -1348,8 +1348,6 @@ namespace json
         class _Value
         {
         public:
-            virtual std::unique_ptr<_Value> _copy_value() = 0;
-
             virtual bool has_field(const utility::string_t &) const { return false; }
             virtual value get_field(const utility::string_t &) const { throw json_exception(_XPLATSTR("not an object")); }
             virtual value get_element(array::size_type) const { throw json_exception(_XPLATSTR("not an array")); }
@@ -1419,18 +1417,7 @@ namespace json
         class _Null : public _Value
         {
         public:
-
-            virtual std::unique_ptr<_Value> _copy_value()
-            {
-                return utility::details::make_unique<_Null>();
-            }
-
             virtual json::value::value_type type() const { return json::value::Null; }
-
-            _Null() { }
-
-        private:
-            template<typename CharType> friend class json::details::JSON_Parser;
         };
 
         class _Number : public _Value
@@ -1442,11 +1429,6 @@ namespace json
             _Number(int64_t value) : m_number(value) { }
             _Number(uint64_t value) : m_number(value) { }
 
-            virtual std::unique_ptr<_Value> _copy_value()
-            {
-                return utility::details::make_unique<_Number>(*this);
-            }
-
             virtual json::value::value_type type() const { return json::value::Number; }
 
             virtual bool is_integer() const { return m_number.is_integral(); }
@@ -1478,11 +1460,7 @@ namespace json
         class _Boolean : public _Value
         {
         public:
-
-            virtual std::unique_ptr<_Value> _copy_value()
-            {
-                return utility::details::make_unique<_Boolean>(*this);
-            }
+            _Boolean(bool value) : m_value(value) { }
 
             virtual json::value::value_type type() const { return json::value::Boolean; }
 
@@ -1502,9 +1480,6 @@ namespace json
 #endif
         private:
             template<typename CharType> friend class json::details::JSON_Parser;
-        public:
-            _Boolean(bool value) : m_value(value) { }
-        private:
             bool m_value;
         };
 
@@ -1545,11 +1520,6 @@ namespace json
                 return *this;
             }
 
-            virtual std::unique_ptr<_Value> _copy_value()
-            {
-                return utility::details::make_unique<_String>(*this);
-            }
-
             virtual json::value::value_type type() const { return json::value::String; }
 
             virtual const utility::string_t & as_string() const;
@@ -1627,11 +1597,6 @@ namespace json
 
             virtual ~_Object() {}
 
-            virtual std::unique_ptr<_Value> _copy_value()
-            {
-                return utility::details::make_unique<_Object>(*this);
-            }
-
             virtual json::object& as_object() { return m_object;    }
 
             virtual const json::object& as_object() const { return m_object;    }
@@ -1740,11 +1705,6 @@ namespace json
             _Array(array::size_type size) : m_array(size) {}
             _Array(array::storage_type elements) : m_array(std::move(elements)) { }
 
-            virtual std::unique_ptr<_Value> _copy_value()
-            {
-                return utility::details::make_unique<_Array>(*this);
-            }
-
             virtual json::value::value_type type() const { return json::value::Array; }
 
             virtual json::array& as_array() { return m_array; }

Release/src/json/json.cpp

@@ -123,7 +123,7 @@ m_value(utility::details::make_unique<web::json::details::_String>(utility::stri
 { }
 
 web::json::value::value(const value &other) :
-    m_value(other.m_value->_copy_value())
+    m_value(other.m_value.get())
 #ifdef ENABLE_JSON_VALUE_VISUALIZER
     ,m_kind(other.m_kind)
 #endif
@@ -133,7 +133,7 @@ web::json::value &web::json::value::operator=(const value &other)
 {
     if(this != &other)
     {
-        m_value = std::unique_ptr<details::_Value>(other.m_value->_copy_value());
+        m_value = std::unique_ptr<details::_Value>(other.m_value.get());
 #ifdef ENABLE_JSON_VALUE_VISUALIZER
         m_kind = other.m_kind;
 #endif