Skip to content

Commit 7b61141

Browse files
author
Antoine Brunner
committed
Fix a bug in lastIndexOf and lastIndexWhere
1 parent b8bdbf3 commit 7b61141

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

library/src-bootstrapped/scala/IArray.scala

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -122,11 +122,11 @@ object opaques
122122
def [T](arr: IArray[T]) lastOption: Option[T] =
123123
genericArrayOps(arr).lastOption
124124

125-
def [T](arr: IArray[T]) lastIndexOf(elem: T, from: Int = 0): Int =
126-
genericArrayOps(arr.asInstanceOf[Array[T]]).lastIndexOf(elem, from)
125+
def [T](arr: IArray[T]) lastIndexOf(elem: T, end: Int = arr.length - 1): Int =
126+
genericArrayOps(arr.asInstanceOf[Array[T]]).lastIndexOf(elem, end)
127127

128-
def [T](arr: IArray[T]) lastIndexWhere(p: T => Boolean, from: Int = 0): Int =
129-
genericArrayOps(arr).lastIndexWhere(p, from)
128+
def [T](arr: IArray[T]) lastIndexWhere(p: T => Boolean, end: Int = arr.length - 1): Int =
129+
genericArrayOps(arr).lastIndexWhere(p, end)
130130

131131
def [T, U: ClassTag](arr: IArray[T]) map(f: T => U): IArray[U] =
132132
genericArrayOps(arr).map(f).asInstanceOf[IArray[U]]

tests/run-bootstrapped/iarray-extmtds.scala

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -70,9 +70,9 @@ object Test extends App {
7070

7171
println(arr2.lastOption)
7272

73-
println(arr2.lastIndexOf(17)) // Might be a bug
73+
println(arr2.lastIndexOf(17))
7474

75-
println(arr1.lastIndexWhere(_ < 5)) // Might be a bug
75+
println(arr1.lastIndexWhere(_ < 5))
7676

7777
assertDifferent(arr1.map(_ + 10), arr1)
7878

0 commit comments

Comments
 (0)