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

Modified future behaviour when completing with an exception.

Browse source 
Futures will now raise the exception if they did not have a callback
associated with them.
This commit is contained in:
Dominik Picheta 2014-05-24 15:51:41 +01:00
commit d54b902441
1 changed files with 5 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

5
lib/pure/asyncdispatch.nim
View file

@ -71,6 +71,11 @@ proc fail*[T](future: PFuture[T], error: ref EBase) =
future.error = error future.error = error
if future.cb != nil: if future.cb != nil:
future.cb() future.cb()
else:
# This is to prevent exceptions from being silently ignored when a future
# is discarded.
# TODO: This may turn out to be a bad idea.
raise error
proc `callback=`*(future: PFutureBase, cb: proc () {.closure,gcsafe.}) = proc `callback=`*(future: PFutureBase, cb: proc () {.closure,gcsafe.}) =
## Sets the callback proc to be called when the future completes. ## Sets the callback proc to be called when the future completes.