From 4ca963a68258abed7f158ffa5289030026e11720 Mon Sep 17 00:00:00 2001
From: eaon <eaon@mit.edu>
Date: Wed, 21 Nov 2018 14:06:21 -0500
Subject: Return Result<(), Status::NotmuchError> instead of Status

---
 src/database.rs | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

(limited to 'src/database.rs')

diff --git a/src/database.rs b/src/database.rs
index 5ed9537..102a7a9 100644
--- a/src/database.rs
+++ b/src/database.rs
@@ -7,7 +7,7 @@ use supercow::Supercow;
 
 use libc;
 
-use error::Result;
+use error::{Error, Result};
 use ffi;
 use ffi::Status;
 use query::QueryPtr;
@@ -235,7 +235,7 @@ impl Database {
         <Self as DatabaseExt>::all_tags(self)
     }
 
-    pub fn remove_message<'d, P>(&'d self, path: &P) -> Status
+    pub fn remove_message<'d, P>(&'d self, path: &P) -> Result<()>
     where
         P: AsRef<Path>,
     {
@@ -288,7 +288,7 @@ pub trait DatabaseExt {
         }
     }
 
-    fn remove_message<'d, D, P>(database: D, path: &P) -> Status
+    fn remove_message<'d, D, P>(database: D, path: &P) -> Result<()>
     where
         D: Into<Supercow<'d, Database>>,
         P: AsRef<Path>,
@@ -298,11 +298,10 @@ pub trait DatabaseExt {
             Some(path_str) => {
                 let msg_path = CString::new(path_str).unwrap();
 
-                Status::from(unsafe {
-                    ffi::notmuch_database_remove_message(dbref.handle.ptr, msg_path.as_ptr())
-                })
+                unsafe { ffi::notmuch_database_remove_message(dbref.handle.ptr, msg_path.as_ptr()) }
+                    .as_result()
             }
-            None => Status::FileError,
+            None => Err(Error::NotmuchError(Status::FileError)),
         }
     }
 }
-- 
cgit v1.2.1