Aaand I'm a moron, who managed to screw up posting a trivial patch.
This is the version that actually works:
(the first was missing a # on body-result):
Index: src/clojure/contrib/sql/sql.clj
===================================================================
--- src/clojure/contrib/sql/sql.clj (revision 161)
+++ src/clojure/contrib/sql/sql.clj (working copy)
@@ -31,8 +31,9 @@
`(with-open ~con ~init
(try
(.setAutoCommit ~con false))
- [EMAIL PROTECTED]
- (.commit ~con)
+ (let [body-result# (do [EMAIL PROTECTED])]
+ (.commit ~con)
+ body-result#)
(catch Exception e#
(.rollback ~con)
(throw (Exception. "transaction rolled back" e#)))))
meredydd wrote:
> Hey all,
> The current version of the (with-connection) macro in clojure-
> contrib doesn't return the result of its body. (It just returns nil
> from the commit() call at the end of the transaction). I attach a
> patch to achieve this behaviour, so it can be used in a functional
> style.
>
> Meredydd
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Clojure" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/clojure?hl=en
-~----------~----~----~----~------~----~------~--~---
