technomancy / slamhound

Slamhound rips your namespace form apart and reconstructs it.
Other
473 stars 38 forks source link

Slamhound.el does not use nREPL connection. #32

Open orivej opened 11 years ago

orivej commented 11 years ago

Slamhound.el tests if nREPL is connected with (get-buffer-process nrepl-current-connection-buffer) which is not an adequate test: the variable may be nil even with active connection. nrepl.el 0.1.7 defines a function nrepl-current-connection-buffer as

(defun nrepl-current-connection-buffer ()
  "The connection to use for nREPL interaction."
  (or nrepl-connection-dispatch
      nrepl-connection-buffer
      (first (nrepl-connection-buffers))))

which may be used instead of nrepl-current-connection-buffer in the call to get-buffer-process.