On 13 Feb 2010, at 19:03, Richard Newman wrote: >> The above thread suggests defining *err* as a PrintWriter instead of >> as a Writer. Has this been patched, and is it official? If so, I'll >> patch clojure-swank to use PrintWriter. If not, I'll patch >> clojure.contrib.sql to not use println. > > I patched swank-clojure: > > diff --git a/src/swank/core/server.clj b/src/swank/core/server.clj > index 31c25ee..972e74a 100644 > --- a/src/swank/core/server.clj > +++ b/src/swank/core/server.clj > @@ -57,7 +57,7 @@ > > (defn- socket-serve [connection-serve socket opts] > (with-connection (accept-authenticated-connection socket opts) > - (let [out-redir (make-output-redirection *current-connection*)] > + (let [out-redir (java.io.PrintWriter. (make-output-redirection > *current-connection*))] > (binding [*out* out-redir > *err* out-redir] > (dosync (ref-set (*current-connection* :writer-redir) *out*)) > > It would be nice to see this upstream.
+1. Github user 'wilig' appears to have been working on this (or a very similar issue): http://github.com/wilig/swank-clojure/commits/ Not sure if he's sent a pull request to Phil. -Steve -- 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