From 37531ed7479755a944317c6e2699d125036d7aa2 Mon Sep 17 00:00:00 2001
From: Marco Stronati <marco@stronati.org>
Date: Wed, 21 Aug 2019 08:28:44 +0200
Subject: [PATCH] Fixes after feedback

---
 librustzcash/src/rustzcash.rs | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/librustzcash/src/rustzcash.rs b/librustzcash/src/rustzcash.rs
index 0f47d8f..dca6617 100644
--- a/librustzcash/src/rustzcash.rs
+++ b/librustzcash/src/rustzcash.rs
@@ -143,7 +143,7 @@ pub extern "system" fn librustzcash_init_zksnark_params(
     let output_path = Path::new(OsStr::from_bytes(unsafe {
         slice::from_raw_parts(output_path, output_path_len)
     }));
-    let sprout_path = if sprout_path.is_null() && sprout_path_len == 0 {
+    let sprout_path = if sprout_path.is_null() {
         None
     } else {
         Some(Path::new(OsStr::from_bytes(unsafe {
@@ -178,7 +178,7 @@ pub extern "system" fn librustzcash_init_zksnark_params(
         OsString::from_wide(unsafe { slice::from_raw_parts(spend_path, spend_path_len) });
     let output_path =
         OsString::from_wide(unsafe { slice::from_raw_parts(output_path, output_path_len) });
-    let sprout_path = if sprout_path.is_null() && sprout_path_len == 0 {
+    let sprout_path = if sprout_path.is_null() {
         None
     } else {
         Some(OsStr::from_wide(unsafe {
@@ -215,7 +215,7 @@ fn init_zksnark_params(
         .to_str()
         .expect("hash should be a valid string");
 
-    let sprout_hash_option = if sprout_path.is_none() {
+    let sprout_hash = if sprout_path.is_none() {
         None
     } else {
         Some(
@@ -232,7 +232,7 @@ fn init_zksnark_params(
         output_path,
         output_hash,
         sprout_path,
-        sprout_hash_option,
+        sprout_hash,
     );
 
     // Caller is responsible for calling this function once, so