clippy code cleanup (#81)

* clippy code cleanup

* more cleanups

Author: gkorland

src/alloc.rs

@@ -8,10 +8,10 @@ pub struct RedisAlloc;
 unsafe impl GlobalAlloc for RedisAlloc {
     unsafe fn alloc(&self, layout: Layout) -> *mut u8 {
         let size = (layout.size() + layout.align() - 1) & (!(layout.align() - 1));
-        return raw::RedisModule_Alloc.unwrap()(size) as *mut u8;
+        raw::RedisModule_Alloc.unwrap()(size) as *mut u8
     }
 
     unsafe fn dealloc(&self, ptr: *mut u8, _layout: Layout) {
-        return raw::RedisModule_Free.unwrap()(ptr as *mut c_void);
+        raw::RedisModule_Free.unwrap()(ptr as *mut c_void)
     }
 }

src/error.rs

@@ -1,4 +1,3 @@
-use std;
 use std::error;
 use std::fmt;
 use std::fmt::Display;

src/key.rs

@@ -274,9 +274,7 @@ fn verify_type(key_inner: *mut raw::RedisModuleKey, redis_type: &RedisType) -> R
         let raw_type = unsafe { raw::RedisModule_ModuleTypeGetType.unwrap()(key_inner) };
 
         if raw_type != *redis_type.raw_type.borrow() {
-            return Err(RedisError::String(format!(
-                "Existing key has wrong Redis type"
-            )));
+            return Err(RedisError::Str("Existing key has wrong Redis type"));
         }
     }
 

src/lib.rs

@@ -47,7 +47,7 @@ pub enum LogLevel {
 fn from_byte_string(byte_str: *const c_char, length: size_t) -> Result<String, Utf8Error> {
     let mut vec_str: Vec<u8> = Vec::with_capacity(length as usize);
     for j in 0..length {
-        let byte = unsafe { *byte_str.offset(j as isize) } as u8;
+        let byte = unsafe { *byte_str.add(j) } as u8;
         vec_str.insert(j, byte);
     }