fix(error): Specialize the self-conflicts error

Inspired by rust-lang/cargo#11159
clap-rs · Sep 29, 2022 · 4e9f3cc · 4e9f3cc
1 parent 2c09781
commit 4e9f3cc
Show file tree

Hide file tree

Showing 4 changed files with 27 additions and 21 deletions.
diff --git a/examples/tutorial_builder/03_01_flag_bool.md b/examples/tutorial_builder/03_01_flag_bool.md
@@ -17,7 +17,7 @@ verbose: true
 
 $ 03_01_flag_bool --verbose --verbose
 ? failed
-error: The argument '--verbose' cannot be used with '--verbose'
+error: The argument '--verbose' was provided more than once, but cannot be used multiple times
 
 Usage: 03_01_flag_bool[EXE] [OPTIONS]
 

diff --git a/examples/tutorial_derive/03_01_flag_bool.md b/examples/tutorial_derive/03_01_flag_bool.md
@@ -17,7 +17,7 @@ verbose: true
 
 $ 03_01_flag_bool_derive --verbose --verbose
 ? failed
-error: The argument '--verbose' cannot be used with '--verbose'
+error: The argument '--verbose' was provided more than once, but cannot be used multiple times
 
 Usage: 03_01_flag_bool_derive[EXE] [OPTIONS]
 

diff --git a/src/error/format.rs b/src/error/format.rs
@@ -90,26 +90,32 @@ fn write_dynamic_context(error: &crate::error::Error, styled: &mut StyledStr) ->
             if let (Some(ContextValue::String(invalid_arg)), Some(prior_arg)) =
                 (invalid_arg, prior_arg)
             {
-                styled.none("The argument '");
-                styled.warning(invalid_arg);
-                styled.none("' cannot be used with");
+                if ContextValue::String(invalid_arg.clone()) == *prior_arg {
+                    styled.none("The argument '");
+                    styled.warning(invalid_arg);
+                    styled.none("' was provided more than once, but cannot be used multiple times");
+                } else {
+                    styled.none("The argument '");
+                    styled.warning(invalid_arg);
+                    styled.none("' cannot be used with");
 
-                match prior_arg {
-                    ContextValue::Strings(values) => {
-                        styled.none(":");
-                        for v in values {
-                            styled.none("\n");
-                            styled.none(TAB);
-                            styled.warning(&**v);
+                    match prior_arg {
+                        ContextValue::Strings(values) => {
+                            styled.none(":");
+                            for v in values {
+                                styled.none("\n");
+                                styled.none(TAB);
+                                styled.warning(&**v);
+                            }
+                        }
+                        ContextValue::String(value) => {
+                            styled.none(" '");
+                            styled.warning(value);
+                            styled.none("'");
+                        }
+                        _ => {
+                            styled.none(" one or more of the other specified arguments");
                         }
-                    }
-                    ContextValue::String(value) => {
-                        styled.none(" '");
-                        styled.warning(value);
-                        styled.none("'");
-                    }
-                    _ => {
-                        styled.none(" one or more of the other specified arguments");
                     }
                 }
                 true

diff --git a/tests/builder/conflicts.rs b/tests/builder/conflicts.rs
@@ -313,7 +313,7 @@ For more information try '--help'
 #[cfg(feature = "error-context")]
 fn conflict_output_repeat() {
     static ERR: &str = "\
-error: The argument '-F' cannot be used with '-F'
+error: The argument '-F' was provided more than once, but cannot be used multiple times
 
 Usage: clap-test [OPTIONS] [positional] [positional2] [positional3]... [COMMAND]