Skip to content

Commit 0feb70a

Browse files
author
Judd
committed
fix for test cases
1 parent a3b256d commit 0feb70a

File tree

1 file changed

+8
-9
lines changed

1 file changed

+8
-9
lines changed

lib/pure/collections/sequtils.nim

+8-9
Original file line numberDiff line numberDiff line change
@@ -142,7 +142,7 @@ func concat*[T](seqs: varargs[seq[T]]): seq[T] =
142142
inc(i)
143143

144144
func addUnique*[T](s: var seq[T], x: sink T) =
145-
## Adds `x` to the container `s` if it is not already present.
145+
## Adds `x` to the container `s` if it is not already present.
146146
## Uses `==` to check if the item is already present.
147147
runnableExamples:
148148
var a = @[1, 2, 3]
@@ -920,10 +920,10 @@ template foldl*(sequence, operation: untyped): untyped =
920920
assert concatenation == "nimiscool"
921921
assert foldl(procs, foo(a, b)) == "procIsAlsoFine"
922922

923-
assert sequence.len > 0, "Can't fold empty sequences"
924-
var result: typeof(sequence[0])
925-
result = sequence[0]
926-
for i in 1..<sequence.len:
923+
let n = sequence.len
924+
assert n > 0, "Can't fold empty sequences"
925+
var result = sequence[0]
926+
for i in 1..<n:
927927
let
928928
a {.inject.} = result
929929
b {.inject.} = sequence[i]
@@ -988,13 +988,12 @@ template foldr*(sequence, operation: untyped): untyped =
988988
assert multiplication == 495, "Multiplication is (5*(9*(11)))"
989989
assert concatenation == "nimiscool"
990990

991-
let s = sequence # xxx inefficient, use {.evalonce.} pending #13750
992-
let n = s.len
991+
let n = sequence.len
993992
assert n > 0, "Can't fold empty sequences"
994-
var result = s[n - 1]
993+
var result = sequence[n - 1]
995994
for i in countdown(n - 2, 0):
996995
let
997-
a {.inject.} = s[i]
996+
a {.inject.} = sequence[i]
998997
b {.inject.} = result
999998
result = operation
1000999
result

0 commit comments

Comments
 (0)