update

Author: dota17

include/json/value.h

@@ -467,7 +467,7 @@ Json::Value obj_value(Json::objectValue); // {}
   bool insert(ArrayIndex index, const Value& newValue);
 #if JSON_HAS_RVALUE_REFERENCES
   Value& append(Value&& value);
-  bool insert(ArrayIndex index, const Value&& newValue);
+  bool insert(ArrayIndex index, Value&& newValue);
 #endif
 
   /// Access an object value by name, create a null member if it does not exist.

src/lib_json/json_value.cpp

@@ -1182,8 +1182,9 @@ bool Value::insert(ArrayIndex index, const Value& newValue) {
   resize(oldsize + 1);
   ArrayIndex length = size();
 
-  for(ArrayIndex i = length ; i> index ; i--) {
-    (*this)[i] = (*this)[i-1];
+  for (ArrayIndex i = length; i > index; i--) {
+    CZString key(i);
+    (*value_.map_)[key] = (*this)[i - 1];
   }
   (*this)[index] = newValue;
   return true;
@@ -1192,7 +1193,7 @@ bool Value::insert(ArrayIndex index, const Value& newValue) {
 Value& Value::append(Value&& value) {
   return (*this)[size()] = std::move(value);
 }
-bool Value::insert(ArrayIndex index, const Value&& newValue) {
+bool Value::insert(ArrayIndex index, Value&& newValue) {
   if (type() != arrayValue) {
     return false;
   }
@@ -1203,8 +1204,9 @@ bool Value::insert(ArrayIndex index, const Value&& newValue) {
   resize(oldsize + 1);
   ArrayIndex length = size();
 
-  for(ArrayIndex i = length ; i> index ; i--) {
-    (*this)[i] = (*this)[i-1];
+  for (ArrayIndex i = length; i > index; i--) {
+    CZString key(i);
+    (*value_.map_)[key] = std::move((*this)[i - 1]);
   }
   (*this)[index] = std::move(newValue);
   return true;

src/test_lib_json/main.cpp

@@ -313,30 +313,30 @@ JSONTEST_FIXTURE(ValueTest, arrayIssue252) {
 JSONTEST_FIXTURE(ValueTest, arrayInsertAtRandomIndex) {
   Json::Value array;
   JSONCPP_STRING str = "index5";
-  array.append("index0");  
-  array.append("index1");  
-  array.append("index2");  
-  array.append("index3");  
-
-  JSONTEST_ASSERT_EQUAL(Json::Value("index0"),array[0]);
-  JSONTEST_ASSERT_EQUAL(Json::Value("index1"),array[1]);
-  JSONTEST_ASSERT_EQUAL(Json::Value("index2"),array[2]);
-  JSONTEST_ASSERT_EQUAL(Json::Value("index3"),array[3]);
-
-  array.insert(3,"index4"); //rvalue
-  JSONTEST_ASSERT_EQUAL(Json::Value("index0"),array[0]);
-  JSONTEST_ASSERT_EQUAL(Json::Value("index1"),array[1]);    
-  JSONTEST_ASSERT_EQUAL(Json::Value("index2"),array[2]);    
-  JSONTEST_ASSERT_EQUAL(Json::Value("index4"),array[3]);    
-  JSONTEST_ASSERT_EQUAL(Json::Value("index3"),array[4]);    
-
-  array.insert(4,str); //lvalue
-  JSONTEST_ASSERT_EQUAL(Json::Value("index0"),array[0]);
-  JSONTEST_ASSERT_EQUAL(Json::Value("index1"),array[1]);    
-  JSONTEST_ASSERT_EQUAL(Json::Value("index2"),array[2]);    
-  JSONTEST_ASSERT_EQUAL(Json::Value("index4"),array[3]);    
-  JSONTEST_ASSERT_EQUAL(Json::Value("index5"),array[4]);    
-  JSONTEST_ASSERT_EQUAL(Json::Value("index3"),array[5]);    
+  array.append("index0");
+  array.append("index1");
+  array.append("index2");
+  array.append("index3");
+
+  JSONTEST_ASSERT_EQUAL(Json::Value("index0"), array[0]);
+  JSONTEST_ASSERT_EQUAL(Json::Value("index1"), array[1]);
+  JSONTEST_ASSERT_EQUAL(Json::Value("index2"), array[2]);
+  JSONTEST_ASSERT_EQUAL(Json::Value("index3"), array[3]);
+
+  array.insert(3, "index4"); // rvalue
+  JSONTEST_ASSERT_EQUAL(Json::Value("index0"), array[0]);
+  JSONTEST_ASSERT_EQUAL(Json::Value("index1"), array[1]);
+  JSONTEST_ASSERT_EQUAL(Json::Value("index2"), array[2]);
+  JSONTEST_ASSERT_EQUAL(Json::Value("index4"), array[3]);
+  JSONTEST_ASSERT_EQUAL(Json::Value("index3"), array[4]);
+
+  array.insert(4, str); // lvalue
+  JSONTEST_ASSERT_EQUAL(Json::Value("index0"), array[0]);
+  JSONTEST_ASSERT_EQUAL(Json::Value("index1"), array[1]);
+  JSONTEST_ASSERT_EQUAL(Json::Value("index2"), array[2]);
+  JSONTEST_ASSERT_EQUAL(Json::Value("index4"), array[3]);
+  JSONTEST_ASSERT_EQUAL(Json::Value("index5"), array[4]);
+  JSONTEST_ASSERT_EQUAL(Json::Value("index3"), array[5]);
 }
 JSONTEST_FIXTURE(ValueTest, null) {
   JSONTEST_ASSERT_EQUAL(Json::nullValue, null_.type());