sort: fix parsing of end field in obsolescent key formats
authorPádraig Brady <P@draigBrady.com>
Tue, 20 Apr 2010 15:25:55 +0000 (16:25 +0100)
committerPádraig Brady <P@draigBrady.com>
Tue, 20 Apr 2010 21:37:18 +0000 (22:37 +0100)
commit8fc12909f649fbe75eb84c48ace554ae3e4e6c06
tree34cadaffde2978c7033fae2ce54355330d7b1159
parent1777d0dfe34dc4d8c148a34a96eb92c6036ff7bd
sort: fix parsing of end field in obsolescent key formats

This regression was introduced in commit 224a69b5, 2009-02-24,
"sort: Fix two bugs with determining the end of field".
The specific regression being that we include 1 field too many when
an end field is specified using obsolescent key syntax (+POS -POS).

* src/sort.c (struct keyfield): Clarify the description of the eword
member, as suggested by Alan Curry.
(main): When processing obsolescent format key specifications,
normalize eword to a zero based count when no specific end char is given
for an end field. This matches what's done when keys are specified with -k.
* tests/misc/sort: Add a few more tests for the obsolescent key formats,
with test 07i being the particular failure addressed by this change.
* THANKS: Add Alan Curry who precisely identified the issue.
* NEWS: Mention the fix.
Reported by Santiago Rodríguez
NEWS
THANKS
src/sort.c
tests/misc/sort