bhauman / bhauman/bhauman.github.com

A question about queue behavior in todos_async/ex2.cljs

Open
#2 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
15
Forks
9
PR merge metrics
No merged PRs in 30d

Description

Hi Bruce,

If the function [merge](https://github.com/bhauman/bhauman.github.com/blob/master/assets/cljs/todos-async/chan-utils.cljs#L34-L40) try inside a loop to remove/take all the values of all the queues/channels passed to it to fill in with these values the merged channel, then it has no much sense try try to access [again](https://github.com/bhauman/bhauman.github.com/blob/master/assets/cljs/todos-async/ex2.cljs#L25) to one of these channels while the merge running [is already running](https://github.com/bhauman/bhauman.github.com/blob/master/assets/cljs/todos-async/ex2.cljs#L21) the commented loop

Also I have seen that in ex4.cljs you only use the merged-channel to swith the state mode so maybe it is an improved use of the channels ...

Thanks in advance for sharing and your welcomed clojure/async posts
Juan

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.