[Fix #429] Last occurrence sometimes not replaced for move-to-let (#430)

In the following case:

* there are more than one occurrences of an expression
* and `move-to-let` is not initiated from the last occurrence
* and the actual bound name is longer than the expression being moved to `let`

the last expression won't be replaced.

The solution: the end of the `let` expression is not cached before calling `clojure--replace-sexps-with-binding`.

Author: benedekfazekas

clojure-mode-refactor-let-test.el

@@ -164,6 +164,21 @@
   (search-backward "(or ")
   (clojure--move-to-let-internal "status"))
 
+(def-refactor-test test-move-to-let-name-longer-than-expression
+  "(defn handle-request
+  (let []
+    (println \"body: \" body \", params: \" \", status: \" 5)
+    {:body body
+     :status 5}))"
+  "(defn handle-request
+  (let [status 5]
+    (println \"body: \" body \", params: \" \", status: \" status)
+    {:body body
+     :status status}))"
+  (search-backward "5")
+  (search-backward "5")
+  (clojure--move-to-let-internal "status"))
+
 ;; clojure-emacs/clj-refactor.el#41
 (def-refactor-test test-move-to-let-nested-scope
   "(defn foo []