This looks fine.  I just don't understand enough clojure to see how it returns a value.