diff options
| author | Jim Meyering <jim@meyering.net> | 2007-05-22 14:25:19 +0200 |
|---|---|---|
| committer | Jim Meyering <jim@meyering.net> | 2007-05-22 18:47:17 +0200 |
| commit | a6a447fc58a01598682f0914f978d0a3c1cfc4dc (patch) | |
| tree | 381e6cc2fecc1d4244daf27a8d2a67eccaf7c0df /src/cut.c | |
| parent | "cut -f 2-0" now fails; before, it was equivalent to "cut -f 2-" (diff) | |
| download | coreutils-a6a447fc58a01598682f0914f978d0a3c1cfc4dc.tar.gz coreutils-a6a447fc58a01598682f0914f978d0a3c1cfc4dc.zip | |
cut: diagnose a range starting with 0 (-f 0-2) as invalid, and
give a better diagnostic for a field-number/offset of 0.
* NEWS: Mention the fix.
* src/cut.c (ADD_RANGE_PAIR): Add an explicit check for 0.
Based on a patch from James Youngman.
* tests/misc/cut: Add tests for the above.
Diffstat (limited to 'src/cut.c')
| -rw-r--r-- | src/cut.c | 2 |
1 files changed, 2 insertions, 0 deletions
@@ -57,6 +57,8 @@ #define ADD_RANGE_PAIR(rp, low, high) \ do \ { \ + if (low == 0 || high == 0) \ + FATAL_ERROR (_("fields and positions are numbered from 1")); \ if (n_rp >= n_rp_allocated) \ { \ (rp) = X2NREALLOC (rp, &n_rp_allocated); \ |
