fix: replace unwraps with Result

Author: icecreammatt

src/ffmpeg.rs

@@ -1,4 +1,4 @@
-use std::{fs, process::Command};
+use std::{fs, io, process::Command};
 
 fn run_command(command: &str) {
     let cmd = Command::new("sh")
@@ -20,7 +20,7 @@ fn run_command(command: &str) {
     println!("{}", result);
 }
 
-pub fn create_preview_gif(input: &str, subpath: &str, output: &str) {
+pub fn create_preview_gif(input: &str, subpath: &str, output: &str) -> io::Result<()> {
     let path = format!("{}/{}", output, subpath);
     if fs::metadata(&path).is_err() {
         fs::create_dir_all(&path).unwrap();
@@ -31,9 +31,10 @@ pub fn create_preview_gif(input: &str, subpath: &str, output: &str) {
     );
 
     run_command(&command);
+    Ok(())
 }
 
-pub fn create_preview_image(input: &str, subpath: &str, output: &str) {
+pub fn create_preview_image(input: &str, subpath: &str, output: &str) -> io::Result<()> {
     let path = format!("{}/{}", output, subpath);
     if fs::metadata(&path).is_err() {
         fs::create_dir_all(&path).unwrap();
@@ -45,9 +46,10 @@ pub fn create_preview_image(input: &str, subpath: &str, output: &str) {
     );
 
     run_command(&command);
+    Ok(())
 }
 
-pub fn create_thumbnails(input: &str, subpath: &str, output: &str) {
+pub fn create_thumbnails(input: &str, subpath: &str, output: &str) -> io::Result<()> {
     println!("\ncreate_thumbnails");
 
     let thumbs_path = format!("{}/{}", output, subpath);
@@ -63,9 +65,10 @@ pub fn create_thumbnails(input: &str, subpath: &str, output: &str) {
     );
 
     run_command(&command);
+    Ok(())
 }
 
-pub fn create_hls_encoding(input: &str, subpath: &str, output: &str) {
+pub fn create_hls_encoding(input: &str, subpath: &str, output: &str) -> io::Result<()> {
     println!("\npub create_hls_encoding");
 
     let hls_path = format!("{}/{}", output, subpath);
@@ -96,4 +99,5 @@ pub fn create_hls_encoding(input: &str, subpath: &str, output: &str) {
         -hls_playlist_type {}/vod \
         {}/720p_%03d.m3u8 {}/720p.m3u8", input, hls_path, hls_path, hls_path);
     run_command(&command);
+    Ok(())
 }

src/main.rs

@@ -1,6 +1,6 @@
 use crate::ffmpeg::{create_preview_gif, create_preview_image};
 use clap::{App, Arg};
-use std::process;
+use std::{io, process};
 
 enum Flags {
     Gif,
@@ -52,7 +52,7 @@ mod fileio;
 mod media_info;
 mod metadata;
 
-fn main() {
+fn main() -> io::Result<()> {
     let matches = App::new("tape-encoder")
         .version("1.0")
         .author("Matt Carrier")
@@ -108,22 +108,22 @@ fn main() {
 
     let gen_thumbs = matches.is_present(Flags::Thumbnails.as_str());
     if gen_thumbs {
-        create_thumbnails(&out.file_name, "thumbs", &path);
+        create_thumbnails(&out.file_name, "thumbs", &path)?;
     }
 
     let gen_preview_image = matches.is_present(Flags::PreviewImage.as_str());
     if gen_preview_image {
-        create_preview_image(&out.file_name, "lg", &path);
+        create_preview_image(&out.file_name, "lg", &path)?;
     }
 
     let gen_hls = matches.is_present(Flags::Hls.as_str());
     if gen_hls {
-        create_hls_encoding(&out.file_name, "hls", &path);
+        create_hls_encoding(&out.file_name, "hls", &path)?;
     }
 
     let gen_gif = matches.is_present(Flags::Gif.as_str());
     if gen_gif {
-        create_preview_gif(&out.file_name, "gif", &path);
+        create_preview_gif(&out.file_name, "gif", &path)?;
     }
 
     /*
@@ -147,4 +147,5 @@ fn main() {
     [ ] Create queue watcher to start running jobs (watches queueu every 5 seconds)
         - On new item run generator process
     */
+    Ok(())
 }