Merge pull request #105 from Syn-McJ/fix/coinjoin-coderabbit-fixes

fix(coinjoin): coderabbit review

Author: Syn-McJ

dash-spv-coinjoin/src/coinjoin_client_manager.rs

@@ -221,8 +221,8 @@ impl CoinJoinClientManager {
                 // (DashJ) we may not need this
                 if !dry_run && self.is_waiting_for_new_block() {
                     let current_time = SystemTime::now().duration_since(UNIX_EPOCH).unwrap().as_secs();
-                            
-                    if current_time - self.last_time_report_too_recent > 15_000 {
+                    
+                    if current_time - self.last_time_report_too_recent > 15 {
                         self.str_auto_denom_result = "Last successful action was too recent.".to_string();
                         log_info!(target: "CoinJoin", "do_automatic_denominating {}", self.str_auto_denom_result);
                         self.last_time_report_too_recent = current_time;

dash-spv-coinjoin/src/masternode_meta_data_manager.rs

@@ -41,10 +41,17 @@ impl MasternodeMetadataManager {
     }
 
     pub fn allow_mixing(&mut self, pro_tx_hash: UInt256) {
-        if let Some(mut mm) = self.get_meta_info(pro_tx_hash, true) {
+        if let Some(mm) = self.meta_infos.get_mut(&pro_tx_hash) {
             self.dsq_count += 1;
             mm.last_dsq = self.dsq_count;
             mm.mixing_tx_count = 0;
+            return;
         }
+
+        let mut info = MasternodeMetaInfo::new(pro_tx_hash);
+        self.dsq_count += 1;
+        info.last_dsq = self.dsq_count;
+        info.mixing_tx_count = 0;
+        self.meta_infos.insert(pro_tx_hash, info);
     }
 }