Make git-add -i accept ranges like 7-

ciaranm · gitster · commit 69c231f4732e · 2008-08-01T23:26:40.000-07:00
git-add -i ranges expect number-number. But for the supremely lazy, typing in
that second number when selecting "from patch 7 to the end" is wasted effort.
So treat an empty second number in a range as "until the last item".

Signed-off-by: Ciaran McCreesh &lt;ciaran.mccreesh@googlemail.com&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
diff --git a/Documentation/git-add.txt b/Documentation/git-add.txt
@@ -187,8 +187,9 @@ update::
    "Update>>".  When the prompt ends with double '>>', you can
    make more than one selection, concatenated with whitespace or
    comma.  Also you can say ranges.  E.g. "2-5 7,9" to choose
-   2,3,4,5,7,9 from the list.  You can say '*' to choose
-   everything.
+   2,3,4,5,7,9 from the list.  If the second number in a range is
+   omitted, all remaining patches are taken.  E.g. "7-" to choose
+   7,8,9 from the list.  You can say '*' to choose everything.
 +
 What you chose are then highlighted with '*',
 like this:
diff --git a/git-add--interactive.perl b/git-add--interactive.perl
@@ -394,9 +394,9 @@ sub list_and_choose {
 			if ($choice =~ s/^-//) {
 				$choose = 0;
 			}
-			# A range can be specified like 5-7
-			if ($choice =~ /^(\d+)-(\d+)$/) {
-				($bottom, $top) = ($1, $2);
+			# A range can be specified like 5-7 or 5-.
+			if ($choice =~ /^(\d+)-(\d*)$/) {
+				($bottom, $top) = ($1, length($2) ? $2 : 1 + @stuff);
 			}
 			elsif ($choice =~ /^\d+$/) {
 				$bottom = $top = $choice;

Original file line number	Diff line number	Diff line change
`@@ -394,9 +394,9 @@ sub list_and_choose {`
`394`	`394`	`if ($choice =~ s/^-//) {`
`395`	`395`	`$choose = 0;`
`396`	`396`	`}`
`397`		`- # A range can be specified like 5-7`
`398`		`- if ($choice =~ /^(\d+)-(\d+)$/) {`
`399`		`- ($bottom, $top) = ($1, $2);`
	`397`	`+ # A range can be specified like 5-7 or 5-.`
	`398`	`+ if ($choice =~ /^(\d+)-(\d*)$/) {`
	`399`	`+ ($bottom, $top) = ($1, length($2) ? $2 : 1 + @stuff);`
`400`	`400`	`}`
`401`	`401`	`elsif ($choice =~ /^\d+$/) {`
`402`	`402`	`$bottom = $top = $choice;`