diff options
author | Thomas Rast <trast@inf.ethz.ch> | 2013-04-05 16:34:48 +0200 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2013-04-05 10:39:09 -0700 |
commit | 209618860c2627a4e134a15472587c574b328b40 (patch) | |
tree | c0c33f379053fb4150b1dbfc1fd506a925c948a4 /revision.h | |
parent | log -L: check range set invariants when we look it up (diff) | |
download | tgif-209618860c2627a4e134a15472587c574b328b40.tar.xz |
log -L: fix overlapping input ranges
The existing code was too defensive, and would trigger the assert in
range_set_append() if the user gave overlapping ranges.
The intent was always to define overlapping ranges as just the union
of all of them, as evidenced by the call to sort_and_merge_range_set().
(Which was already used, unlike what the comment said.)
Fix by splitting out the meat of range_set_append() to a new _unsafe()
function that lacks the paranoia. sort_and_merge_range_set will fix
up the ranges, so we don't need the checks there.
Signed-off-by: Thomas Rast <trast@inf.ethz.ch>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'revision.h')
0 files changed, 0 insertions, 0 deletions