asyncdispatch.sleepAsync broken

[juancarlospaco]

• https://github.com/nim-lang/Nim/blob/devel/lib/pure/asyncdispatch.nim#L1833

sleepAsync takes float (???), it should have same proc signature than sleep and use Natural argument only.

import asyncdispatch, times

proc main(a: float or int) {.async.} =
  let t = now()
  echo "BEFORE\T", t, '\T', a
  await sleepAsync(a)
  echo "AFTER\T", t - now(), '\n'

waitFor main(1)  # Control
waitFor main(0.0)
waitFor main(-0.0)
waitFor main(1.0)
waitFor main(-1.0)
waitFor main(-1.0)
waitFor main(NaN)
waitFor main(+Inf)
waitFor main(-Inf)
waitFor main(Inf)

[ringabout]

see #617

for a better solution

[juancarlospaco]

I agree, but still this needs to be Deprecated, or remove the float overload, or something...