diff options
author | Steven Schubiger <schubiger@gmail.com> | 2008-02-18 22:39:22 +0100 |
---|---|---|
committer | Jim Meyering <meyering@redhat.com> | 2008-02-19 15:41:16 +0100 |
commit | 4f3ff2e7007a6591294e1a656782b5f239f9840c (patch) | |
tree | 7ed8264c7566377c6730cddaaf3b47567e73151b /tests | |
parent | a1e715698a038af7ff341011a2aeecf6729c8de9 (diff) | |
download | coreutils-4f3ff2e7007a6591294e1a656782b5f239f9840c.tar.xz |
seq: give better diagnostics for invalid formats.
* src/seq.c: (validate_format): New function.
(main): Use it.
* tests/misc/seq (fmt-d, fmt-e): Test for expected diagnostics with
invalid formats.
* NEWS: Mention this change.
* TODO: Remove this item.
[jm: src/seq.c: make diagnostics more consistent
tests/misc/seq (fmt-eos1): adjust the expected diagnostic ]
Diffstat (limited to 'tests')
-rwxr-xr-x | tests/misc/seq | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/tests/misc/seq b/tests/misc/seq index 1a153a310..f48289bdf 100755 --- a/tests/misc/seq +++ b/tests/misc/seq @@ -87,10 +87,14 @@ my @Tests = # "seq: memory exhausted". In coreutils-6.0..6.8, it would mistakenly # succeed and print a blank line. ['fmt-eos1', qw(-f % 1), {EXIT => 1}, - {ERR => "seq: invalid format string: `%'\n" . $try_help }], + {ERR => "seq: no % directive in format string `%'\n" . $try_help }], ['fmt-eos2', qw(-f %g% 1), {EXIT => 1}, {ERR => "seq: invalid format string: `%g%'\n" . $try_help }], + ['fmt-d', qw(-f "" 1), {EXIT => 1}, + {ERR => "seq: no % directive in format string `'\n" . $try_help }], + ['fmt-e', qw(-f %g%g 1), {EXIT => 1}, + {ERR => "seq: too many % directives in format string `%g%g'\n"}], ); # Append a newline to each entry in the OUT array. |