sink parameters passed by value and moved from, more flexible, easier to maintain and *nearly* just as efficient (2

moves instead of 1 sometimes, but ... i doubt that will make a difference)

Author: eteran

include/cpp-json/value.h

@@ -58,10 +58,8 @@ class value {
 	~value();
 
 public:
-	explicit value(const object_pointer &o);
-	explicit value(const array_pointer &a);
-	explicit value(object_pointer &&o);
-	explicit value(array_pointer &&a);
+	explicit value(object_pointer o);
+	explicit value(array_pointer a);
 
 public:
 	value(const value &other);

include/cpp-json/value.tcc

@@ -61,14 +61,14 @@ inline value &value::operator=(value &&rhs) {
 //------------------------------------------------------------------------------
 // Name: value
 //------------------------------------------------------------------------------
-inline value::value(object_pointer &&o) : type_(type_object) {
+inline value::value(object_pointer o) : type_(type_object) {
 	new (&value_) object_pointer(std::move(o));
 }
 
 //------------------------------------------------------------------------------
 // Name: value
 //------------------------------------------------------------------------------
-inline value::value(array_pointer &&a) : type_(type_array) {
+inline value::value(array_pointer a) : type_(type_array) {
 	new (&value_) array_pointer(std::move(a));
 }
 
@@ -86,20 +86,6 @@ inline value::value(std::string s) : type_(type_string) {
 	new (&value_) std::string(std::move(s));
 }
 
-//------------------------------------------------------------------------------
-// Name: value
-//------------------------------------------------------------------------------
-inline value::value(const object_pointer &o) : type_(type_object) {
-	new (&value_) object_pointer(o);
-}
-
-//------------------------------------------------------------------------------
-// Name: value
-//------------------------------------------------------------------------------
-inline value::value(const array_pointer &a) : type_(type_array) {
-	new (&value_) array_pointer(a);
-}
-
 //------------------------------------------------------------------------------
 // Name: value
 //------------------------------------------------------------------------------