Specify value_hint macro for better completion

Fix alexheretic#180
sudotac · Feb 3, 2024 · e884298 · e884298
1 parent fae9ce6
commit e884298
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/command/args.rs b/src/command/args.rs
@@ -6,7 +6,7 @@ pub use encode::*;
 pub use vmaf::*;
 
 use crate::{command::encode::default_output_ext, ffprobe::Ffprobe};
-use clap::Parser;
+use clap::{Parser, ValueHint};
 use std::{
     path::{Path, PathBuf},
     sync::Arc,
@@ -19,7 +19,7 @@ pub struct EncodeToOutput {
     /// Output file, by default the same as input with `.av1` before the extension.
     ///
     /// E.g. if unspecified: -i vid.mkv --> vid.av1.mkv
-    #[arg(short, long)]
+    #[arg(short, long, value_hint = ValueHint::FilePath)]
     pub output: Option<PathBuf>,
 
     /// Set the output ffmpeg audio codec.
@@ -69,7 +69,7 @@ pub struct Sample {
 
     /// Directory to store temporary sample data in.
     /// Defaults to using the input's directory.
-    #[arg(long, env = "AB_AV1_TEMP_DIR")]
+    #[arg(long, env = "AB_AV1_TEMP_DIR", value_hint = ValueHint::DirPath)]
     pub temp_dir: Option<PathBuf>,
 
     /// Extension preference for encoded samples (ffmpeg encoder only).

diff --git a/src/command/args/encode.rs b/src/command/args/encode.rs
@@ -4,7 +4,7 @@ use crate::{
     float::TerseF32,
 };
 use anyhow::ensure;
-use clap::Parser;
+use clap::{Parser, ValueHint};
 use std::{
     collections::HashMap,
     fmt::{self, Write},
@@ -23,7 +23,7 @@ pub struct Encode {
     pub encoder: Encoder,
 
     /// Input video file.
-    #[arg(short, long)]
+    #[arg(short, long, value_hint = ValueHint::FilePath)]
     pub input: PathBuf,
 
     /// Ffmpeg video filter applied to the input before av1 encoding.