Merge pull request #257 from calcit-lang/trivial

handle trivial warnings

Author: NoEgAm

src/bin/calcit_deps.rs

@@ -70,7 +70,7 @@ pub fn main() -> Result<(), String> {
       if version != CALCIT_VERSION {
         eprintln!(
           "[Warn] calcit version mismatch, deps.cirru expected {}, running {}",
-          CALCIT_VERSION, version
+          version, CALCIT_VERSION,
         );
       }
     }

src/builtins/refs.rs

@@ -13,7 +13,9 @@ use crate::{call_stack::CallStackList, runner};
 
 pub(crate) type ValueAndListeners = (Calcit, HashMap<EdnTag, Calcit>);
 
-static REFS_DICT: LazyLock<Mutex<HashMap<Arc<str>, Arc<Mutex<ValueAndListeners>>>>> = LazyLock::new(|| Mutex::new(HashMap::new()));
+type RefListeners = HashMap<Arc<str>, Arc<Mutex<ValueAndListeners>>>;
+
+static REFS_DICT: LazyLock<Mutex<RefListeners>> = LazyLock::new(|| Mutex::new(HashMap::new()));
 
 fn modify_ref(locked_pair: Arc<Mutex<ValueAndListeners>>, v: Calcit, call_stack: &CallStackList) -> Result<(), CalcitErr> {
   let (listeners, prev) = {

src/program/entry_book.rs

@@ -140,11 +140,8 @@ where
   pub fn to_hashmap(&self) -> std::collections::HashMap<Arc<str>, T> {
     let mut res = std::collections::HashMap::with_capacity(self.0.len());
     for piece in &self.0 {
-      match &piece.value {
-        Some(v) => {
-          res.insert(piece.key.to_owned(), v.to_owned());
-        }
-        None => {}
+      if let Some(v) = &piece.value {
+        res.insert(piece.key.to_owned(), v.to_owned());
       }
     }
     res