On Saturday, March 23, 2013 2:09:02 PM UTC+1, Jim foo.bar wrote: > > (defmacro let-timed
[bindings & code] > I would write that as [bindings & body] which is also recommended by http://dev.clojure.org/display/design/Library+Coding+Standards > (let [parts (partition 2 bindings) > names (map first parts) > results (map #(list 'time (second %)) parts)] ;;don't time at > compile-time, just build the timing expression for later use `(let ~(vec (interleave names results)) ;;the new bindings I would write that as `(let [~@(interleave names results)] > ~@code))) > I personally prefer to not give intermediate values a name when they are only used in one place because it makes it easier to see that these values are only used in one place, but your mileage may vary. (Suggestive names can help readability, but in the end you have to read and understand what they refer to anyway) -- -- 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/groups/opt_out.
