jyapayne/Nim
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix tables.CountTable largest and smallest (#15115)

Browse source 
It needs to have len defined first because of the assert .len > 0.  I just moved it up a bit to make them work.
This commit is contained in:
treeform 2020-07-29 20:49:51 -07:00 • committed by GitHub
commit 604f7461d7
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

8
lib/pure/collections/tables.nim
View file

@ -2293,6 +2293,10 @@ proc inc*[A](t: var CountTable[A], key: A, val = 1) =
if val != 0: if val != 0:
insertImpl() insertImpl()
proc len*[A](t: CountTable[A]): int =
## Returns the number of keys in ``t``.
result = t.counter
proc smallest*[A](t: CountTable[A]): tuple[key: A, val: int] = proc smallest*[A](t: CountTable[A]): tuple[key: A, val: int] =
## Returns the ``(key, value)`` pair with the smallest ``val``. Efficiency: O(n) ## Returns the ``(key, value)`` pair with the smallest ``val``. Efficiency: O(n)
## ##
@ -2345,10 +2349,6 @@ proc getOrDefault*[A](t: CountTable[A], key: A; default: int = 0): int =
## is in the table ## is in the table
ctget(t, key, default) ctget(t, key, default)
proc len*[A](t: CountTable[A]): int =
## Returns the number of keys in ``t``.
result = t.counter
proc del*[A](t: var CountTable[A], key: A) {.since: (1, 1).} = proc del*[A](t: var CountTable[A], key: A) {.since: (1, 1).} =
## Deletes ``key`` from table ``t``. Does nothing if the key does not exist. ## Deletes ``key`` from table ``t``. Does nothing if the key does not exist.
## ##