diff options
| author |  Hugo Herbelin <Hugo.Herbelin@inria.fr> | 2018-04-19 20:18:03 +0200 |
|---|---|---|
| committer | Hugo Herbelin <Hugo.Herbelin@inria.fr> | 2018-05-25 14:45:45 +0200 |
| commit | 683ee02cd810c57f051f5b5d5d446dcc950f23f7 (patch) | |
| tree | 00b7d0c6777446cbe26f145c5fa4d1501e847a37 /vernac/vernacentries.ml | |
| parent | d2533da244f770261478ae829167cb3a8ad38038 (diff) | |
An attempt to clarify error message for Arguments needing "rename" flag.
Using a formulation which makes it is clear that no renaming has been
done. See discussion at issue #2987.
Diffstat (limited to 'vernac/vernacentries.ml')
| -rw-r--r-- | vernac/vernacentries.ml | 19 |
1 files changed, 10 insertions, 9 deletions
diff --git a/vernac/vernacentries.ml b/vernac/vernacentries.ml index e1ce4e194..85bc518d5 100644 --- a/vernac/vernacentries.ml +++ b/vernac/vernacentries.ml @@ -1131,15 +1131,16 @@ let vernac_arguments ~atts reference args more_implicits nargs_for_red flags = let names = rename prev_names names in let renaming_specified = Option.has_some !example_renaming in - if !rename_flag_required && not rename_flag then - user_err ~hdr:"vernac_declare_arguments" - (strbrk "To rename arguments the \"rename\" flag must be specified." - ++ spc () ++ - match !example_renaming with - | None -> mt () - | Some (o,n) -> - str "Argument " ++ Name.print o ++ - str " renamed to " ++ Name.print n ++ str "."); + if !rename_flag_required && not rename_flag then begin + let msg = + match !example_renaming with + | None -> + strbrk "To rename arguments the \"rename\" flag must be specified." + | Some (o,n) -> + strbrk "Flag \"rename\" expected to rename " ++ Name.print o ++ + strbrk " into " ++ Name.print n ++ str "." + in user_err ~hdr:"vernac_declare_arguments" msg + end; let duplicate_names = List.duplicates Name.equal (List.filter ((!=) Anonymous) names) |