Merge pull request #185 from plexus/catch-java-lang-error

Report problems of type java.lang.Error back to the nREPL client

Author: expez

CHANGELOG.md

@@ -5,6 +5,7 @@
 ### Bugs fixed
 
 *
+* [#185](https://github.com/clojure-emacs/refactor-nrepl/issues/185) Report throwables of type `Error` instead of swallowing them.
 * [clojure-emacs/clj-refactor.el#330](https://github.com/clojure-emacs/clj-refactor.el/issues/332) `clean-ns` removes imported inner inner classes.
 * [clojure-emacs/clj-refactor.el#330](https://github.com/clojure-emacs/clj-refactor.el/issues/330) `clean-ns` ignores namespaced keywords.
 * [#160](https://github.com/clojure-emacs/refactor-nrepl/issues/160) Make `resolve-missing` find newly defined vars and types (clj). Because of a stale cache, newly added vars or types would not be found. This fix takes into account vars/types added by eval-ing code (rescan affected namespace), and by hotloading dependencies (reset the cache).

src/refactor_nrepl/middleware.clj

@@ -36,6 +36,9 @@
        (transport/send
         ~transport (response-for ~msg :error (.getMessage e#) :status :done)))
      (catch Exception e#
+       (transport/send
+        ~transport (response-for ~msg (err-info e# :refactor-nrepl-error))))
+     (catch Error e#
        (transport/send
         ~transport (response-for ~msg (err-info e# :refactor-nrepl-error))))))