search

technomancy / nrepl-discover

proof-of-concept middleware for auto-discovery of nrepl ops
36 stars 7 forks source link

`nrepl-discover` does not protect itself from over-writing effectively. #2

Open coventry opened 10 years ago

coventry commented 10 years ago

Currently the emacs function nrepl-discover excludes ops with the name nrepl-discover, but the value returned by nrepl.discover/ops just has the name discover, which then results in a new defun for nrepl-discover.

coventry commented 10 years ago

(when (not (string= "nrepl-discover" should be (when (not (string= "discover". The nrepl- component is added only after receipt by the elisp side.

The current behavior is a pain for development, because nrepl-discover ends up overwriting itself with the output from nrepl-discover-command-for.