Skip to content

Commit

Permalink
algorithm: remove deprecated reversed proc
Browse files Browse the repository at this point in the history
- remove deprecated `reversed[T](var openArray[T], int, int): seq[T]`
- removed associated tests
  • Loading branch information
saem committed Aug 6, 2022
1 parent aa11679 commit e3c640b
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 6 deletions.
4 changes: 0 additions & 4 deletions lib/pure/algorithm.nim
Original file line number Diff line number Diff line change
Expand Up @@ -143,10 +143,6 @@ proc reversed*[T](a: openArray[T]): seq[T] {.inline.} =
result.setLen(n)
for i in 0..<n: result[i] = a[n - (i + 1)]

proc reversed*[T](a: openArray[T], first: Natural, last: int): seq[T]
{.inline, deprecated: "use: `reversed(toOpenArray(a, first, last))`".} =
reversed(toOpenArray(a, first, last))

when defined(nimHasEffectsOf):
{.experimental: "strictEffects".}
else:
Expand Down
4 changes: 2 additions & 2 deletions tests/stdlib/talgorithm.nim
Original file line number Diff line number Diff line change
Expand Up @@ -45,8 +45,8 @@ block:
var arr1 = @[0, 1, 2, 3, 4]
doAssert arr1.reversed() == @[4, 3, 2, 1, 0]
for i in 0 .. high(arr1):
doAssert arr1.reversed(0, i) == arr1.reversed()[high(arr1) - i .. high(arr1)]
doAssert arr1.reversed(i, high(arr1)) == arr1.reversed()[0 .. high(arr1) - i]
doAssert arr1[0..i].reversed() == arr1.reversed()[high(arr1) - i .. high(arr1)]
doAssert arr1[i..high(arr1)].reversed() == arr1.reversed()[0 .. high(arr1) - i]

block:
var list = [0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10]
Expand Down

0 comments on commit e3c640b

Please sign in to comment.