Bump version to 0.4.1

The pattern matching moves 'body' and drops it after
creation of the pointer. We must avoid the move, by using 'ref'.

.travis.yml

@@ -0,0 +1,21 @@
+language: rust
+rust:
+- nightly
+- beta
+- stable
+before_script:
+- |
+  pip install 'travis-cargo<0.2' --user &&
+  export PATH=$HOME/.local/bin:$PATH
+script:
+- |
+  travis-cargo build &&
+  travis-cargo test &&
+  travis-cargo --only stable doc
+after_success:
+- travis-cargo --only stable doc-upload
+env:
+  global:
+  - TRAVIS_CARGO_NIGHTLY_FEATURE=""
+  - secure: IjahTjjwmuvig/wTDMwxpk1F3Ywi2d0r17+JmOXHPdbfSfhi4puiTzsOgMjBhFtosTwcvlBeFwwFeTtl9KFNG165xm9Fqbhcez9sx+hS+EWZR/MBPFhKorJlgva0nuH8L1cxDUP+mDkcV/BdXCDeT7ml+y/FqEDAI4N0lwEsVMk=
+  - secure: B8QPcNgwOmbJ8dNJn/p/tE1cLeUOLTo/Oj7nOBkK1tatMgS6yfQFa5pyNEqeTyZAyNptssqSH1BpRC3RxTJ/b+kFzdy8Kq1nkJfp9R9zTY34w5gWukfmmTH4Qe2lyWY/DKL1lGVjb+8mroV9sPaS5Y4DqYHbeYtpWF8Gi27vIL4=

Cargo.toml

@@ -1,8 +1,8 @@
 [package]
 
 name = "libnotify"
-version = "0.0.4"
-authors = ["Mika Attila <radiantstatue@gmail.com>"]
+version = "0.4.1"
+authors = ["Mika Attila <radiantstatue@gmail.com>", "Julian Ospald <hasufell@posteo.de>"]
 license = "MIT"
 description = "Rust bindings to libnotify"
 readme = "README.md"
@@ -11,6 +11,6 @@ documentation = "http://crumblingstatue.github.io/doc/libnotify/libnotify/"
 keywords = ["libnotify", "notification"]
 
 [dependencies]
-libnotify-sys = "*"
-glib-2_0-sys = "*"
-gtypes = "*"
+libnotify-sys = "0.3"
+glib-2-0-sys = "0.46.0"
+gtypes = "0.2"

README.md

@@ -1,2 +1,4 @@
+[![Build Status](https://travis-ci.org/crumblingstatue/rust-libnotify.svg)](https://travis-ci.org/crumblingstatue/rust-libnotify)
+
 # libnotify-rs
 Rust binding to libnotify

examples/show.rs

@@ -1,12 +1,11 @@
 extern crate libnotify;
 
 fn main() {
-    let notify = libnotify::Context::new("hello").unwrap_or_else(|e| {
-        panic!("{}", e);
-    });
-    let body_text = Some("This is the optional body text.");
-    let n = notify.new_notification("This is the summary.",
-                                    body_text,
-                                    None).unwrap_or_else(|e| panic!("{}", e));
-    n.show().ok().expect("Failed to show notification");
+    // Create a libnotify context
+    let notify = libnotify::Context::new("myapp").unwrap();
+    // Create a new notification and show it
+    let n = notify.new_notification("Summary", Some("Optional Body"), None).unwrap();
+    n.show().unwrap();
+    // You can also use the .show() convenience method on the context
+    notify.show("I am another notification", None, None).unwrap();
 }

src/lib.rs

@@ -4,68 +4,91 @@
 //! extern crate libnotify;
 //!
 //! fn main() {
-//!     let notify = libnotify::Context::new("hello").unwrap_or_else(|e| {
-//!         panic!("{}", e);
-//!     });
-//!     let body_text = Some("This is the optional body text.");
-//!     let n = notify.new_notification("This is the summary.",
-//!                                     body_text,
-//!                                     None).unwrap_or_else(|e| {
-//!         panic!("{}", e);
-//!     });
-//!     n.show().ok().expect("Failed to show notification");
+//!     // Create a libnotify context
+//!     let notify = libnotify::Context::new("myapp").unwrap();
+//!     // Create a new notification and show it
+//!     let n = notify.new_notification("Summary",
+//!                                     Some("Optional Body"),
+//!                                     None).unwrap();
+//!     n.show().unwrap();
+//!     // You can also use the .show() convenience method on the context
+//!     notify.show("I am another notification", None, None).unwrap();
 //! }
+//!
 //! ```
 
+#![warn(missing_docs)]
+
 extern crate libnotify_sys as sys;
 extern crate glib_2_0_sys as glib;
 extern crate gtypes;
 
-use std::ffi::CString;
+use std::ffi::{self, CStr, CString};
 use std::marker::PhantomData;
 use std::fmt;
+use std::error::Error;
 
-use gtypes::{
-    TRUE,
-    FALSE
-};
+use gtypes::{TRUE, FALSE};
 
 /// Error that can happen on context creation
 #[derive(Debug)]
 pub enum ContextCreationError {
-    /// Context already exists
+    /// Context already exists.
     AlreadyExists,
-    InitFailure,
-    NulError
+    /// Failed to initialize libnotify.
+    InitError,
+    /// A nul byte was found in the provided string.
+    NulError(ffi::NulError),
 }
 
 impl fmt::Display for ContextCreationError {
-    fn fmt(&self, f: &mut fmt::Formatter) -> Result<(), fmt::Error> {
+    fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
         use ContextCreationError::*;
         match *self {
             AlreadyExists => write!(f, "A Libnotify context already exists."),
-            InitFailure => write!(f, "Failed to initialize libnotify."),
-            NulError => write!(f, "Argument contains a nul character.")
+            InitError => write!(f, "Failed to initialize libnotify."),
+            NulError(ref e) => write!(f, "{}", e),
         }
     }
 }
 
+impl From<ffi::NulError> for ContextCreationError {
+    fn from(src: ffi::NulError) -> Self {
+        ContextCreationError::NulError(src)
+    }
+}
+
 #[derive(Debug)]
+/// An error that can happen when attempting to create a notification.
 pub enum NotificationCreationError {
-    NulError,
-    Unknown
+    /// A nul byte was found in the provided string.
+    NulError(ffi::NulError),
+    /// An unknown error happened.
+    Unknown,
 }
 
 impl fmt::Display for NotificationCreationError {
-    fn fmt(&self, f: &mut fmt::Formatter) -> Result<(), fmt::Error> {
+    fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
         use NotificationCreationError::*;
         match *self {
-            NulError => write!(f, "Argument contains a nul character."),
-            Unknown => write!(f, "Unknown error")
+            NulError(ref e) => write!(f, "{}", e),
+            Unknown => write!(f, "Unknown error"),
         }
     }
 }
 
+impl From<ffi::NulError> for NotificationCreationError {
+    fn from(src: ffi::NulError) -> Self {
+        NotificationCreationError::NulError(src)
+    }
+}
+
+impl Error for NotificationCreationError {
+    fn description(&self) -> &str {
+        "notification creation error"
+    }
+}
+
 /// The context which within libnotify operates.
 ///
 /// Only one context can exist at a time.
@@ -82,12 +105,9 @@ impl Context {
             if sys::notify_is_initted() == TRUE {
                 return Err(ContextCreationError::AlreadyExists);
             }
-            let app_name = match CString::new(app_name) {
-                Ok(name) => name,
-                Err(_) => return Err(ContextCreationError::NulError)
-            };
+            let app_name = try!(CString::new(app_name));
             if sys::notify_init(app_name.as_ptr()) == FALSE {
-                return Err(ContextCreationError::InitFailure);
+                return Err(ContextCreationError::InitError);
             }
         }
         Ok(Context)
@@ -99,47 +119,54 @@ impl Context {
     /// - summary: Required summary text
     /// - body: Optional body text
     /// - icon: Optional icon theme icon name or filename
-    pub fn new_notification(&self, summary: &str,
-                                   body: Option<&str>,
-                                   icon: Option<&str>)
-        -> Result<Notification, NotificationCreationError> {
-        let summary = match CString::new(summary) {
-            Ok(cstr) => cstr,
-            Err(_) => return Err(NotificationCreationError::NulError)
-        };
+    pub fn new_notification(&self,
+                            summary: &str,
+                            body: Option<&str>,
+                            icon: Option<&str>)
+                            -> Result<Notification, NotificationCreationError> {
+        let summary = try!(CString::new(summary));
         let body = match body {
-            Some(body) => match CString::new(body) {
-                Ok(cstr) => Some(cstr),
-                Err(_) => return Err(NotificationCreationError::NulError)
-            },
-            None => None
+            Some(body) => Some(try!(CString::new(body))),
+            None => None,
         };
         let body_ptr = match body {
-            Some(body) => body.as_ptr(),
-            None => std::ptr::null()
+            Some(ref body) => body.as_ptr(),
+            None => std::ptr::null(),
         };
         let icon = match icon {
-            Some(icon) => match CString::new(icon) {
-                Ok(cstr) => Some(cstr),
-                Err(_) => return Err(NotificationCreationError::NulError)
-            },
-            None => None
+            Some(icon) => Some(try!(CString::new(icon))),
+            None => None,
         };
         let icon_ptr = match icon {
-            Some(icon) => icon.as_ptr(),
-            None => std::ptr::null()
+            Some(ref icon) => icon.as_ptr(),
+            None => std::ptr::null(),
         };
+
         unsafe {
-            let n = sys::notify_notification_new(summary.as_ptr(),
-                                                 body_ptr,
-                                                 icon_ptr);
+            let n = sys::notify_notification_new(summary.as_ptr(), body_ptr, icon_ptr);
             if n.is_null() {
                 return Err(NotificationCreationError::Unknown);
             }
 
-            Ok(Notification{handle: n, _phantom: PhantomData})
+            Ok(Notification {
+                handle: n,
+                _phantom: PhantomData,
+            })
         }
     }
+    /// Show a notification.
+    ///
+    /// This is a convenience method that creates a new notification,
+    /// and shows it in one step.
+    pub fn show(&self,
+                summary: &str,
+                body: Option<&str>,
+                icon: Option<&str>)
+                -> Result<(), Box<Error>> {
+        let notif = try!(self.new_notification(summary, body, icon));
+        try!(notif.show());
+        Ok(())
+    }
 }
 
 impl Drop for Context {
@@ -153,21 +180,42 @@ impl Drop for Context {
 /// A passive pop-up notification
 pub struct Notification<'a> {
     handle: *mut sys::NotifyNotification,
-    _phantom: PhantomData<&'a Context>
+    _phantom: PhantomData<&'a Context>,
 }
 
 impl<'a> Notification<'a> {
     /// Tells the notification server to display the notification
     /// on the screen.
-    pub fn show(&'a self) -> Result<(), ()> {
+    pub fn show(&'a self) -> Result<(), NotificationShowError> {
         unsafe {
             let mut err: *mut glib::GError = std::ptr::null_mut();
             sys::notify_notification_show(self.handle, &mut err);
             if !err.is_null() {
+                let result = Err(NotificationShowError {
+                    message: CStr::from_ptr((*err).message).to_string_lossy().into_owned(),
+                });
                 glib::g_error_free(err);
-                return Err(())
+                return result;
             }
-            return Ok(())
+            Ok(())
         }
     }
 }
+
+/// An error that can happen when attempting to show a notification.
+#[derive(Debug)]
+pub struct NotificationShowError {
+    message: String,
+}
+
+impl fmt::Display for NotificationShowError {
+    fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
+        write!(f, "Error showing notification: {}", self.message)
+    }
+}
+
+impl Error for NotificationShowError {
+    fn description(&self) -> &str {
+        "notification show error"
+    }
+}