updated based on PR feedback

.gitignore

@@ -2,10 +2,6 @@
 # will have compiled files and executables
 **/target/
 
-# Remove Cargo.lock from gitignore if creating an executable, leave it for libraries
-# More information here https://doc.rust-lang.org/cargo/guide/cargo-toml-vs-cargo-lock.html
-# Cargo.lock
-
 # These are backup files generated by rustfmt
 **/*.rs.bk
 
@@ -23,4 +19,4 @@ dump.rdb
 .gdb_history
 
 venv/
-tmp/*
+tmp/*

src/context/commands.rs

@@ -454,12 +454,12 @@ api! {[
             // the only CString pointers which are not freed are those of the key_specs, lets free them here.
             key_specs.into_iter().for_each(|v|{
                 if !v.notes.is_null() {
-                    unsafe{let _ = CString::from_raw(v.notes as *mut c_char);};
+                    unsafe{drop(CString::from_raw(v.notes as *mut c_char))};
                 }
                 if v.begin_search_type == raw::RedisModuleKeySpecBeginSearchType_REDISMODULE_KSPEC_BS_KEYWORD {
                     let keyword = unsafe{v.bs.keyword.keyword};
                     if !keyword.is_null() {
-                        unsafe{let _= CString::from_raw(v.bs.keyword.keyword as *mut c_char);};
+                        unsafe{drop(CString::from_raw(v.bs.keyword.keyword as *mut c_char))};
                     }
                 }
             });

src/context/mod.rs

@@ -874,7 +874,7 @@ impl Context {
 
 extern "C" fn post_notification_job_free_callback<F: FnOnce(&Context)>(pd: *mut c_void) {
     unsafe {
-        let _ = Box::from_raw(pd as *mut Option<F>);
+        drop(Box::from_raw(pd as *mut Option<F>));
     };
 }