Bugfix for Interval Set

This is necessary or else the last else if will never be executed and the function returns wrong results for some cases. For example, these tests will not run without this change:

TestRemoveRangeSameStart() {
	i := NewIntervalSet()
	i.addInterval(antlr.NewInterval(1, 10))
	i.removeRange(antlr.NewInterval(1, 7))

	suite.Len(i.intervals, 1)
	suite.Equal(7, i.intervals[0].Start)
	suite.Equal(10, i.intervals[0].Stop)
}

TestRemoveRangeOverStartBound() {
	i := NewIntervalSet()
	i.addInterval(antlr.NewInterval(5, 15))

	i.removeRange(antlr.NewInterval(1, 10))
	suite.Len(i.intervals, 1)
	suite.Equal(10, i.intervals[0].Start)
	suite.Equal(15, i.intervals[0].Stop)
}


Signed-off-by: Gustavo de Morais <[email protected]>

runtime/Go/antlr/v4/interval_set.go

@@ -186,7 +186,7 @@ func (i *IntervalSet) removeRange(v Interval) {
 				//                i.intervals.splice(k, 1)
 				i.intervals = append(i.intervals[0:k], i.intervals[k+1:]...)
 				k = k - 1 // need another pass
-			} else if v.Start < ni.Stop {
+			} else if v.Start < ni.Stop && v.Start > ni.Start {
 				i.intervals[k] = NewInterval(ni.Start, v.Start)
 			} else if v.Stop < ni.Stop {
 				i.intervals[k] = NewInterval(v.Stop, ni.Stop)