Skip to content
This repository was archived by the owner on Apr 20, 2020. It is now read-only.

Commit e0f1555

Browse files
committed
repalce iterator with foreach loop
1 parent 79d48d7 commit e0f1555

File tree

1 file changed

+3
-8
lines changed

1 file changed

+3
-8
lines changed

src/lib.rs

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -88,22 +88,17 @@ fn json_get(ctx: &Context, args: Vec<String>) -> RedisResult {
8888

8989
fn json_mget(ctx: &Context, args: Vec<String>) -> RedisResult {
9090

91-
if args.len() < 2 {
91+
if args.len() < 3 {
9292
return Err(RedisError::WrongArity);
9393
}
9494
if let Some(path) = args.last() {
9595
let mut path = path.clone();
9696
if path.starts_with(".") { // backward compatibility
9797
path.insert(0, '$');
9898
}
99-
let keys_len = args.len()-2;
100-
let mut args_iter = args.into_iter().skip(1);
101-
let mut results: Vec<String> = Vec::with_capacity(keys_len);
102-
for _i in 0..keys_len {
103-
let key = args_iter.next_string()?;
104-
99+
let mut results: Vec<String> = Vec::with_capacity(args.len()-2);
100+
for key in &args[1..args.len()-1] {
105101
let redis_key = ctx.open_key_writable(&key);
106-
107102
match redis_key.get_value::<RedisJSON>(&REDIS_JSON_TYPE)? {
108103
Some(doc) => {
109104
let result = doc.to_string(&path)?;

0 commit comments

Comments
 (0)