Use anyhow

Cargo.lock

@@ -35,6 +35,12 @@ dependencies = [
  "winapi",
 ]
 
+[[package]]
+name = "anyhow"
+version = "1.0.40"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "28b2cd92db5cbd74e8e5028f7e27dd7aa3090e89e4f2a197cc7c8dfb69c7063b"
+
 [[package]]
 name = "atty"
 version = "0.2.14"
@@ -794,6 +800,7 @@ dependencies = [
 name = "taskwarrior-tui"
 version = "0.12.2"
 dependencies = [
+ "anyhow",
  "better-panic",
  "cassowary",
  "chrono",

Cargo.toml

@@ -36,6 +36,7 @@ rustyline = "7.1.0"
 uuid = { version = "0.8.1", features = ["serde", "v4"] }
 better-panic = "0.2.0"
 shellexpand = "2.1"
+anyhow = "1"
 
 [package.metadata.rpm]
 package = "taskwarrior-tui"

src/app.rs

@@ -12,11 +12,9 @@ use crate::util::{Event, Events};
 use std::cmp::Ordering;
 use std::collections::{HashMap, HashSet};
 use std::convert::TryInto;
-use std::error::Error;
 use std::fs;
 use std::path::Path;
 use std::process::Command;
-use std::result::Result;
 use std::time::SystemTime;
 
 use task_hookrs::date::Date;
@@ -30,6 +28,8 @@ use unicode_segmentation::UnicodeSegmentation;
 
 use chrono::{Datelike, Local, NaiveDate, NaiveDateTime, TimeZone};
 
+use anyhow::Result;
+
 use std::sync::{Arc, Mutex};
 use std::{sync::mpsc, thread, time::Duration};
 use tui::{
@@ -170,7 +170,7 @@ pub struct TaskwarriorTuiApp {
 }
 
 impl TaskwarriorTuiApp {
-    pub fn new() -> Result<Self, Box<dyn Error>> {
+    pub fn new() -> Result<Self> {
         let c = Config::default()?;
         let mut kc = KeyConfig::default();
         kc.update()?;
@@ -210,7 +210,7 @@ impl TaskwarriorTuiApp {
         Ok(app)
     }
 
-    pub fn get_context(&mut self) -> Result<(), Box<dyn Error>> {
+    pub fn get_context(&mut self) -> Result<()> {
         let output = Command::new("task").arg("_get").arg("rc.context").output()?;
         self.current_context = String::from_utf8_lossy(&output.stdout).to_string();
         self.current_context = self.current_context.strip_suffix('\n').unwrap_or("").to_string();
@@ -825,7 +825,7 @@ impl TaskwarriorTuiApp {
         (tasks, headers)
     }
 
-    pub fn update(&mut self, force: bool) -> Result<(), Box<dyn Error>> {
+    pub fn update(&mut self, force: bool) -> Result<()> {
         if force || self.tasks_changed_since(self.last_export)? {
             self.last_export = Some(std::time::SystemTime::now());
             self.task_report_table.export_headers()?;
@@ -980,7 +980,7 @@ impl TaskwarriorTuiApp {
         self.current_selection = i;
     }
 
-    pub fn export_contexts(&mut self) -> Result<(), Box<dyn Error>> {
+    pub fn export_contexts(&mut self) -> Result<()> {
         let output = Command::new("task").arg("context").output()?;
         let data = String::from_utf8_lossy(&output.stdout);
 
@@ -1014,7 +1014,7 @@ impl TaskwarriorTuiApp {
         Ok(())
     }
 
-    fn get_task_files_max_mtime(&self) -> Result<SystemTime, Box<dyn Error>> {
+    fn get_task_files_max_mtime(&self) -> Result<SystemTime> {
         let data_dir = shellexpand::tilde(&self.config.data_location).into_owned();
         let mut mtimes = Vec::new();
         for fname in &["backlog.data", "completed.data", "pending.data"] {
@@ -1025,7 +1025,7 @@ impl TaskwarriorTuiApp {
         Ok(*mtimes.iter().max().unwrap())
     }
 
-    pub fn tasks_changed_since(&mut self, prev: Option<SystemTime>) -> Result<bool, Box<dyn Error>> {
+    pub fn tasks_changed_since(&mut self, prev: Option<SystemTime>) -> Result<bool> {
         if let Some(prev) = prev {
             match self.get_task_files_max_mtime() {
                 Ok(mtime) => {
@@ -1047,7 +1047,7 @@ impl TaskwarriorTuiApp {
         }
     }
 
-    pub fn export_tasks(&mut self) -> Result<(), Box<dyn Error>> {
+    pub fn export_tasks(&mut self) -> Result<()> {
         let mut task = Command::new("task");
 
         task.arg("rc.json.array=on");
@@ -1076,9 +1076,10 @@ impl TaskwarriorTuiApp {
         let data = String::from_utf8_lossy(&output.stdout);
         let error = String::from_utf8_lossy(&output.stderr);
         if !error.contains("The expression could not be evaluated.") {
-            let imported = import(data.as_bytes())?;
-            *(self.tasks.lock().unwrap()) = imported;
-            self.tasks.lock().unwrap().sort_by(cmp);
+            if let Ok(imported) = import(data.as_bytes()) {
+                *(self.tasks.lock().unwrap()) = imported;
+                self.tasks.lock().unwrap().sort_by(cmp);
+            }
         }
 
         Ok(())
@@ -1653,7 +1654,7 @@ impl TaskwarriorTuiApp {
         input: Key,
         terminal: &mut Terminal<CrosstermBackend<io::Stdout>>,
         events: &Events,
-    ) -> Result<(), Box<dyn Error>> {
+    ) -> Result<()> {
         match self.mode {
             AppMode::TaskReport => {
                 if input == Key::Esc {

src/config.rs

@@ -1,3 +1,4 @@
+use anyhow::Result;
 use std::collections::HashMap;
 use std::error::Error;
 use std::process::Command;
@@ -58,7 +59,7 @@ pub struct Config {
 }
 
 impl Config {
-    pub fn default() -> Result<Self, Box<dyn Error>> {
+    pub fn default() -> Result<Self> {
         let bool_collection = Self::get_bool_collection();
         Ok(Self {
             enabled: true,
@@ -105,7 +106,7 @@ impl Config {
         Some(Self::get_tcolor(&s))
     }
 
-    fn get_color_collection() -> Result<HashMap<String, Style>, Box<dyn Error>> {
+    fn get_color_collection() -> Result<HashMap<String, Style>> {
         let mut color_collection = HashMap::new();
         let output = Command::new("task").arg("rc.color=off").arg("show").output()?;
 

src/keyconfig.rs

@@ -1,4 +1,6 @@
 use crate::util::Key;
+use anyhow::anyhow;
+use anyhow::Result;
 use serde::{Deserialize, Serialize};
 use std::collections::HashSet;
 use std::error::Error;
@@ -89,7 +91,7 @@ impl Default for KeyConfig {
 }
 
 impl KeyConfig {
-    pub fn update(&mut self) -> Result<(), Box<dyn Error>> {
+    pub fn update(&mut self) -> Result<()> {
         self.quit = self
             .get_config("uda.taskwarrior-tui.keyconfig.quit")
             .unwrap_or(self.quit);
@@ -162,7 +164,7 @@ impl KeyConfig {
         self.check()
     }
 
-    pub fn check(&self) -> Result<(), Box<dyn Error>> {
+    pub fn check(&self) -> Result<()> {
         let mut elements = vec![
             &self.quit,
             &self.refresh,
@@ -196,7 +198,7 @@ impl KeyConfig {
         if l == elements.len() {
             Ok(())
         } else {
-            Err("Duplicate keys found in key config".into())
+            Err(anyhow!("Duplicate keys found in key config"))
         }
     }
 

src/main.rs

@@ -13,6 +13,7 @@ mod task_report;
 mod util;
 
 use crate::util::{destruct_terminal, setup_terminal, Event, EventConfig, Events};
+use anyhow::Result;
 use clap::{App, Arg};
 use std::env;
 use std::error::Error;
@@ -26,7 +27,7 @@ use app::{AppMode, TaskwarriorTuiApp};
 const APP_VERSION: &str = env!("CARGO_PKG_VERSION");
 const APP_NAME: &str = env!("CARGO_PKG_NAME");
 
-fn main() -> Result<(), Box<dyn Error>> {
+fn main() -> Result<()> {
     better_panic::install();
     let matches = App::new(APP_NAME)
         .version(APP_VERSION)
@@ -62,7 +63,7 @@ fn main() -> Result<(), Box<dyn Error>> {
     }
 }
 
-fn tui_main(_config: &str) -> Result<(), Box<dyn Error>> {
+fn tui_main(_config: &str) -> Result<()> {
     // Terminal initialization
     let mut terminal = setup_terminal();
 

src/task_report.rs

@@ -1,3 +1,4 @@
+use anyhow::Result;
 use chrono::{Datelike, Local, NaiveDate, NaiveDateTime, TimeZone};
 use itertools::join;
 use std::error::Error;
@@ -42,7 +43,7 @@ pub struct TaskReportTable {
 }
 
 impl TaskReportTable {
-    pub fn new() -> Result<Self, Box<dyn Error>> {
+    pub fn new() -> Result<Self> {
         let virtual_tags = vec![
             "PROJECT",
             "BLOCKED",
@@ -89,7 +90,7 @@ impl TaskReportTable {
         Ok(task_report_table)
     }
 
-    pub fn export_headers(&mut self) -> Result<(), Box<dyn Error>> {
+    pub fn export_headers(&mut self) -> Result<()> {
         self.columns = vec![];
         self.labels = vec![];