Use new function in implementation of list subcommand

neithernut · neithernut · commit e4b8ef7f8fa6 · 2018-05-05T17:56:40.000+02:00
diff --git a/src/main.rs b/src/main.rs
@@ -317,24 +317,16 @@ fn list_impl(matches: &clap::ArgMatches) {
         issues.truncate(str::parse(number).unwrap_or_abort());
     }
 
-    // spawn a pager
-    let mut pager = system::programs::pager(repo.config().unwrap_or_abort())
-        .unwrap_or_abort();
-
-    issues
+    // present the list to the user
+    let result = issues
         .into_iter()
         .map(|issue| issue.initial_message())
         .abort_on_err()
         .flat_map(|initial| formatter.iter().formatted_lines(initial))
         .abort_on_err()
-        .write_lines(pager.stdin.as_mut().unwrap())
+        .pipe_lines(repo.pager())
         .unwrap_or_abort();
-
-    // don't trash the shell by exitting with a child still printing to it
-    let result = pager.wait().unwrap_or_abort();
-    if !result.success() {
-        std::process::exit(result.code().unwrap_or(1));
-    }
+    std::process::exit(result);
 }
 
 
