fix review comments

Author: gkorland

src/lib.rs

@@ -1,7 +1,7 @@
 #[macro_use]
 extern crate redismodule;
 
-use redismodule::{Context, RedisResult, NextArg, RedisValue};
+use redismodule::{Context, RedisResult, NextArg};
 use redismodule::native_types::RedisType;
 
 mod redisjson;
@@ -55,7 +55,7 @@ fn json_strlen(ctx: &Context, args: Vec<String>) -> RedisResult {
     let key = ctx.open_key_writable(&key);
 
     let length = match key.get_value::<RedisJSON>(&REDIS_JSON_TYPE)? {
-        Some(doc) => RedisValue::Integer(doc.str_len(&path)? as i64),
+        Some(doc) => doc.str_len(&path)?.into(),
         None => ().into()
     };
 

src/redisjson.rs

@@ -62,13 +62,7 @@ impl RedisJSON {
 
     pub fn str_len(&self, path: &str) -> Result<usize, Error> {
         let s = match self.get_doc(path)? {
-            Some(doc) => {
-                if doc.is_string() {
-                    serde_json::to_string(&doc)?.len() - 2 // removes ""
-                } else {
-                    0 // the value is not a String
-                }
-            }
+            Some(doc) => doc.as_str().map_or(0, |d| d.len()),
             None => 0 // path not found
         };
         Ok(s)