On Fri, 2012-08-31 at 05:08 -0700, shaobohou wrote:
> I have written the following function using take-while and a pred
> function with an atom to store the set of unique tokens. It works

Only because in the current implementation, take-while happens to be
called in coll order, which is not a guarantee of the API.

> Will this be problem in my implementation if I am calling it from
> multiple threads?

No.

> And what would be the more idiomatic way of implementing this kind of
> behaviour?

Consider this restructuring:

(->> ss (map tok-fn), (reductions (partial apply merge) #{}),
     (take-while #(...)), (map #(do %2 %1) ss), last)

Use of `atom' for this sort of thing is certainly an antipattern, so
consider any alternative that suits you.

-- 
Stephen Compall
"^aCollection allSatisfy: [:each | aCondition]": less is better than


-- 
You received this message because you are subscribed to the Google
Groups "Clojure" group.
To post to this group, send email to clojure@googlegroups.com
Note that posts from new members are moderated - please be patient with your 
first post.
To unsubscribe from this group, send email to
clojure+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/clojure?hl=en

Reply via email to