2014-04-12 16:18 GMT+02:00 Fergal Byrne <fergalbyrnedub...@gmail.com>:

> That's fine, but note that creating new Calendar objects has an overhead,
> using System.currentTimeMillis() is a static OS call which your now()
> probably uses, as well as object creation. It probably won't be millisecond
> sized, but you can measure your (now) using criterium.bench to be sure
> .
>

OK, I'll look into a rewrite.



> Also note that reduce in the last line is lazy by default, so ensure to
> wrap it in a doall or vec to make it greedy.
>

The doseq also good I think?

-- 
Cecil Westerhof

-- 
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
--- 
You received this message because you are subscribed to the Google Groups 
"Clojure" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to clojure+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to