replace std::Result with anyhow::Result

Signed-off-by: he1pa <18012015693@163.com>
kcl-lang · Nov 1, 2024 · 0961bb7 · 0961bb7
1 parent ffded1d
commit 0961bb7
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/kclvm/Cargo.lock b/kclvm/Cargo.lock
diff --git a/kclvm/ast/Cargo.toml b/kclvm/ast/Cargo.toml
@@ -15,6 +15,7 @@ kclvm-span = { path = "../span" }
 kclvm-error = { path = "../error" }
 thread_local = "1.1.7"
 kclvm-utils = {path = "../utils"}
+anyhow = "1.0"
 
 [dev-dependencies]
 kclvm-parser = { path = "../parser" }
diff --git a/kclvm/ast/src/ast.rs b/kclvm/ast/src/ast.rs
@@ -452,11 +452,11 @@ impl Program {
     pub fn get_module(
         &self,
         module_path: &str,
-    ) -> Result<Option<RwLockReadGuard<'_, Module>>, &str> {
+    ) -> anyhow::Result<Option<RwLockReadGuard<'_, Module>>> {
         match self.modules.get(module_path) {
             Some(module_ref) => match module_ref.read() {
                 Ok(m) => Ok(Some(m)),
-                Err(_) => Err("Failed to acquire module lock"),
+                Err(_) => Err(anyhow::anyhow!("Failed to acquire module lock")),
             },
             None => Ok(None),
         }
@@ -465,11 +465,11 @@ impl Program {
     pub fn get_module_mut(
         &self,
         module_path: &str,
-    ) -> Result<Option<RwLockWriteGuard<'_, Module>>, &str> {
+    ) -> anyhow::Result<Option<RwLockWriteGuard<'_, Module>>> {
         match self.modules.get(module_path) {
             Some(module_ref) => match module_ref.write() {
                 Ok(m) => Ok(Some(m)),
-                Err(_) => Err("Failed to acquire module lock"),
+                Err(_) => Err(anyhow::anyhow!("Failed to acquire module lock")),
             },
             None => Ok(None),
         }