Merge pull request #98 from konstin/master

Add `array_remove` to `JsonValue`

Author: maciejhirsz

src/value/mod.rs

@@ -476,6 +476,22 @@ impl JsonValue {
         }
     }
 
+    /// Works on `JsonValue::Array` - remove an entry and return the value it held.
+    /// If the method is called on anything but an object or if the index is out of bounds, it
+    /// will return `JsonValue::Null`.
+    pub fn array_remove(&mut self, index: usize) -> JsonValue {
+        match *self {
+            JsonValue::Array(ref mut vec) => {
+                if index < vec.len() {
+                    vec.remove(index)
+                } else {
+                    JsonValue::Null
+                }
+            },
+            _ => JsonValue::Null
+        }
+    }
+
     /// When called on an array or an object, will wipe them clean. When called
     /// on a string will clear the string. Numbers and booleans become null.
     pub fn clear(&mut self) {

tests/value.rs

@@ -157,6 +157,16 @@ fn array_pop() {
     assert_eq!(data, array![1, 2]);
 }
 
+#[test]
+fn array_remove() {
+    let mut data = array![1, 2, 3];
+
+    assert_eq!(data.array_remove(1), 2);
+    assert_eq!(data, array![1, 3]);
+    // Test with index out of bounds
+    assert_eq!(data.array_remove(2), JsonValue::Null);
+}
+
 #[test]
 fn array_members() {
     let data = array![1, "foo"];