Fix #4392 and progress on #4170 #4393

Closed
wants to merge 1 commit into
from

Projects

None yet

1 participant

@edubart
Contributor
edubart commented Jun 22, 2016

I do not expect to be the most correct fix, but this is my workaround for now, this should make await throw exceptions for futures not just procs, and also remove the last "read" call that await was doing after the yield statement for this specific case, ping @dom96.

All tests under async, incluing tasyncsemantics still pass, although new tests could be made.

@scriptum scriptum pushed a commit to scriptum/Nim that referenced this pull request Jun 22, 2016
@Varriount Varriount Fix #4393 6dcaafc
@edubart edubart closed this Jun 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment