Merge pull request #4128 from epage/default

fix(derive): Remove error case for default_value
clap-rs · Aug 26, 2022 · 2f4e42f · 2f4e42f
2 parents c13b824 + 4e6733f
commit 2f4e42f
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 35 deletions.
diff --git a/clap_derive/src/parse.rs b/clap_derive/src/parse.rs
@@ -161,12 +161,6 @@ impl Parse for ClapAttr {
                 "external_subcommand" => Ok(ExternalSubcommand(name)),
                 "verbatim_doc_comment" => Ok(VerbatimDocComment(name)),
 
-                "default_value" => {
-                    abort!(name,
-                        "`#[clap(default_value)` attribute (without a value) has been replaced by `#[clap(default_value_t)]`.";
-                        help = "Change the attribute to `#[clap(default_value_t)]`";
-                    )
-                }
                 "default_value_t" => Ok(DefaultValueT(name, None)),
                 "default_value_os_t" => Ok(DefaultValueOsT(name, None)),
                 "about" => (Ok(About(name))),

diff --git a/tests/derive_ui/default_value_wo_value_removed.rs b/tests/derive_ui/default_value_wo_value_removed.rs
diff --git a/tests/derive_ui/default_value_wo_value_removed.stderr b/tests/derive_ui/default_value_wo_value_removed.stderr