change function for building ExtractorOpts object in order to make ar…

…range the interface same with ArchiverOpts

src/extractor.rs

@@ -17,6 +17,18 @@ pub struct ExtractorOpts {
 }
 
 impl ExtractorOpts {
+    pub fn new(opts: &CliOpts) -> ExtractorOpts {
+        let d = opts.output.clone();
+        ExtractorOpts {
+            dest: d.unwrap_or_else(|| {
+                PathBuf::from(".")
+            }),
+            use_archive_name_dir: opts.to_archive_name_dir,
+            overwrite: opts.overwrite,
+            v: create_verboser(opts.verbose),
+        }
+    }
+
     pub fn destination(&self, target: &PathBuf) -> PathBuf {
         if self.use_archive_name_dir {
             let file_name = target.file_name().unwrap().to_str().unwrap();
@@ -36,18 +48,6 @@ pub trait Extractor {
     fn format(&self) -> Format;
 }
 
-pub fn create_extract_opts(opts: &CliOpts) -> ExtractorOpts {
-    let d = opts.output.clone();
-    ExtractorOpts {
-        dest: d.unwrap_or_else(|| {
-            PathBuf::from(".")
-        }),
-        use_archive_name_dir: opts.to_archive_name_dir,
-        overwrite: opts.overwrite,
-        v: create_verboser(opts.verbose),
-    }
-}
-
 pub fn create_extractor(file: &PathBuf) -> Result<Box<dyn Extractor>> {
     let format = find_format(file.file_name());
     match format {

src/main.rs

@@ -2,7 +2,7 @@ use clap::Parser;
 use cli::*;
 use cli::{ToatError, RunMode};
 use archiver::{archiver_info, ArchiverOpts};
-use extractor::{create_extract_opts, extractor_info};
+use extractor::{extractor_info, ExtractorOpts};
 
 mod cli;
 mod format;
@@ -32,7 +32,7 @@ fn perform(mut opts: CliOpts) -> Result<()> {
 
 fn perform_extract(opts: CliOpts) -> Result<()> {
     let args = opts.args.clone();
-    let extract_opts = create_extract_opts(&opts);
+    let extract_opts = ExtractorOpts::new(&opts);
     for arg in args.iter() {
         let extractor = extractor::create_extractor(arg).unwrap();
         let target = arg.to_path_buf();