Saturate rc to -1

Seulgi Kim · sgkim126 · commit 329f6bb27792 · 2019-12-11T21:44:53.000+09:00
diff --git a/util/memorydb/src/lib.rs b/util/memorydb/src/lib.rs
@@ -161,6 +161,9 @@ impl MemoryDB {
                 *old_value = value;
             }
             *old_rc += rc;
+            if *old_rc < -1 {
+                *old_rc = -1;
+            }
         }
     }
 }
@@ -279,6 +282,6 @@ mod tests {
 
         assert_eq!(overlay[&remove_key], (b"doggo".to_vec(), 0));
         assert_eq!(overlay[&insert_key], (b"arf".to_vec(), 2));
-        assert_eq!(overlay[&negative_remove_key], (b"negative".to_vec(), -2));
+        assert_eq!(overlay[&negative_remove_key], (b"negative".to_vec(), -1));
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -161,6 +161,9 @@ impl MemoryDB {`
`161`	`161`	`*old_value = value;`
`162`	`162`	`}`
`163`	`163`	`*old_rc += rc;`
	`164`	`+ if *old_rc < -1 {`
	`165`	`+ *old_rc = -1;`
	`166`	`+ }`
`164`	`167`	`}`
`165`	`168`	`}`
`166`	`169`	`}`
`@@ -279,6 +282,6 @@ mod tests {`
`279`	`282`
`280`	`283`	`assert_eq!(overlay[&remove_key], (b"doggo".to_vec(), 0));`
`281`	`284`	`assert_eq!(overlay[&insert_key], (b"arf".to_vec(), 2));`
`282`		`- assert_eq!(overlay[&negative_remove_key], (b"negative".to_vec(), -2));`
	`285`	`+ assert_eq!(overlay[&negative_remove_key], (b"negative".to_vec(), -1));`
`283`	`286`	`}`
`284`	`287`	`}`