code-reviews

Jakub Holý 2015-07-31T06:42:29.001088Z

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

2015-07-31T09:43:53.001090Z

@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ý 2015-07-31T09:52:32.001091Z

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.

2015-07-31T09:53:10.001092Z

Thanks!