[Bug #20725] Should not call compare on `nil`-endpoint

It means unbounded, always inclusive of other ranges.
This commit is contained in:
Nobuyoshi Nakada 2024-09-13 10:50:38 +09:00
Родитель 0f3dc2f958
Коммит 24493779b7
Не найден ключ, соответствующий данной подписи
Идентификатор ключа GPG: 3582D74E1FEE4465
2 изменённых файлов: 2 добавлений и 1 удалений

Просмотреть файл

@ -2477,7 +2477,7 @@ range_overlap(VALUE range, VALUE other)
/* if both begin values are equal, no more comparisons needed */
if (rb_cmpint(cmp, self_beg, other_beg) == 0) return Qtrue;
}
else if (NIL_P(self_beg) && NIL_P(other_beg)) {
else if (NIL_P(self_beg) && !NIL_P(self_end) && NIL_P(other_beg)) {
VALUE cmp = rb_funcall(self_end, id_cmp, 1, other_end);
return RBOOL(!NIL_P(cmp));
}

Просмотреть файл

@ -1471,6 +1471,7 @@ class TestRange < Test::Unit::TestCase
assert_operator((..3), :overlap?, (3..))
assert_operator((nil..nil), :overlap?, (3..))
assert_operator((nil...nil), :overlap?, (nil..))
assert_operator((nil..nil), :overlap?, (..3))
assert_raise(TypeError) { (1..3).overlap?(1) }