Fork me on GitHub
#code-reviews
<
2015-07-31
>
Jakub Holý (HolyJak)06:07:29

@jamesmintram: I am afraid the intentions behind your example aren't clear enough to be able to provide a meaningful input.

jamesmintram09:07:53

@holyjak: Thanks for taking a look - the purpose of the ALTs code is to detect when the Channel is full - instead of it blocking when we try to send the event through it. That particular technique I saw describe as an answer to this: http://tgk.github.io/2013/10/inspect-core-async-channels.html#comment-1084973602 So the async/alts!! call will return :failed when the channel is full - I just wondered if there was a better way to detect and respond to this than using (if (=

Jakub Holý (HolyJak)09:07:32

Oh, thanks for the explanation, that is good to know! And I should have read alts! docstring better. I do not see if as bad there, after all you are doing a decision.