get_possible_values

clap_generate/src/generators/shells/bash.rs

@@ -180,7 +180,7 @@ fn option_details_for_path(app: &App, path: &str) -> String {
 fn vals_for(o: &Arg) -> String {
     debugln!("Bash::vals_for: o={}", o.name);
 
-    if let Some(ref vals) = o.possible_vals {
+    if let Some(ref vals) = o.get_possible_values() {
         format!("$(compgen -W \"{}\" -- ${{cur}})", vals.join(" "))
     } else {
         String::from("$(compgen -f ${cur})")

clap_generate/src/generators/shells/fish.rs

@@ -69,7 +69,7 @@ fn gen_fish_inner(root_command: &str, app: &App, buffer: &mut String) {
             template.push_str(format!(" -d '{}'", escape_string(data)).as_str());
         }
 
-        if let Some(ref data) = option.possible_vals {
+        if let Some(ref data) = option.get_possible_values() {
             template.push_str(format!(" -r -f -a \"{}\"", data.join(" ")).as_str());
         }
 

clap_generate/src/generators/shells/zsh.rs

@@ -379,7 +379,7 @@ fn write_opts_of(p: &App) -> String {
             ""
         };
 
-        let pv = if let Some(ref pv_vec) = o.possible_vals {
+        let pv = if let Some(ref pv_vec) = o.get_possible_values() {
             format!(
                 ": :({})",
                 pv_vec
@@ -504,8 +504,7 @@ fn write_positionals_of(p: &App) -> String {
                 .replace("]", "\\]")
                 .replace(":", "\\:"),
             action = arg
-                .possible_vals
-                .as_ref()
+                .get_possible_values()
                 .map_or("_files".to_owned(), |values| {
                     format!(
                         "({})",

src/build/arg/mod.rs

@@ -72,8 +72,7 @@ pub struct Arg<'help> {
     pub(crate) aliases: Option<Vec<(&'help str, bool)>>, // (name, visible)
     pub(crate) disp_ord: usize,
     pub(crate) unified_ord: usize,
-    #[doc(hidden)]
-    pub possible_vals: Option<Vec<&'help str>>,
+    pub(crate) possible_vals: Option<Vec<&'help str>>,
     pub(crate) val_names: Option<VecMap<&'help str>>,
     pub(crate) num_vals: Option<u64>,
     pub(crate) max_vals: Option<u64>,
@@ -105,15 +104,20 @@ impl<'help> Arg<'help> {
         self.help.clone()
     }
 
-    /// Get the short option for this argument, if any
+    /// Get the short option name for this argument, if any
     pub fn get_short(&self) -> Option<char> {
         self.short.clone()
     }
 
-    /// Get the long option for this argument, if any
+    /// Get the long option name for this argument, if any
     pub fn get_long(&self) -> Option<&str> {
         self.long.clone()
     }
+
+    /// Get the list of the possible values for this argument, if any
+    pub fn get_possible_values(&self) -> Option<&[&str]> {
+        self.possible_vals.as_deref()
+    }
 }
 
 impl<'help> Arg<'help> {