diff options
author | apense <apense@users.noreply.github.com> | 2015-06-17 20:14:53 -0400 |
---|---|---|
committer | apense <apense@users.noreply.github.com> | 2015-06-17 20:14:53 -0400 |
commit | 34910471419637bc616c08f369715ffbf4d881d1 (patch) | |
tree | 1ced89e530dfa30de6f7bb6bd724bca8fdb8aea1 | |
parent | ea1809a931d401e97959f8fd3ddd44b5f65c77c0 (diff) | |
download | Nim-34910471419637bc616c08f369715ffbf4d881d1.tar.gz |
Fixed silly continue
The old if/else was weird and unnecessary
-rw-r--r-- | lib/pure/algorithm.nim | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/lib/pure/algorithm.nim b/lib/pure/algorithm.nim index bfc2e0351..32bf9eb98 100644 --- a/lib/pure/algorithm.nim +++ b/lib/pure/algorithm.nim @@ -240,14 +240,11 @@ template sortedByIt*(seq1, op: expr): expr = proc isSorted*[T](a: openarray[T], cmp: proc(x, y: T): int {.closure.}, order = SortOrder.Ascending): bool = - ## Tests whether `a` is sorted - if len(a) <= 1: return true # empty or one-element lists are already sorted + if len(a) <= 1: return true result = true for i in 0..<len(a)-1: - if cmp(a[i],a[i+1]) * order <= 0: # same test as `sort` - continue - else: + if cmp(a[i],a[i+1]) * order > 0: return false proc product*[T](x: openArray[seq[T]]): seq[seq[T]] = |