diff options
| author | Eric Sunshine <sunshine@sunshineco.com> | 2013-07-31 04:15:45 -0400 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2013-08-05 11:54:32 -0700 |
| commit | 5d57cac6ae7c661d430dcc7dd2e44c994bf797be (patch) | |
| tree | 96f4ff8a5bbeba74d42a5ec4619782acedfd82f0 /line-range.c | |
| parent | 82cd7e5d3e56c173769bc5fba91e0c6415e208f0 (diff) | |
| download | git-5d57cac6ae7c661d430dcc7dd2e44c994bf797be.tar.gz | |
blame: reject empty ranges -L,+0 and -L,-0es/blame-L-more
Empty ranges -L,+0 and -L,-0 are nonsensical in the context of blame yet
they are accepted (in fact, both are interpreted as -L1,Y where Y is
end-of-file). Report them as invalid.
Signed-off-by: Eric Sunshine <sunshine@sunshineco.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'line-range.c')
| -rw-r--r-- | line-range.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/line-range.c b/line-range.c index a816951e06..69e8d6b6c0 100644 --- a/line-range.c +++ b/line-range.c @@ -21,7 +21,7 @@ static const char *parse_loc(const char *spec, nth_line_fn_t nth_line, * for 20 lines, or "-L <something>,-5" for 5 lines ending at * <something>. */ - if (1 < begin && (spec[0] == '+' || spec[0] == '-')) { + if (1 <= begin && (spec[0] == '+' || spec[0] == '-')) { num = strtol(spec + 1, &term, 10); if (term != spec + 1) { if (!ret) |