Skip to content

Commit 2c5b733

Browse files
authored
Rename a variable (#874)
1 parent 458efba commit 2c5b733

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

src/cider/nrepl/middleware/refresh.clj

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -76,8 +76,8 @@
7676
msg]
7777
(reload-utils/after-reply error msg))
7878

79-
(def ^{:added "0.48.0"} user-indicated-clear?
80-
"Whether the user indicated that a `clear` is desired."
79+
(def ^{:added "0.48.0"} client-requested-clear?
80+
"Whether the nREPL client indicated that a `clear` is desired."
8181
(atom false))
8282

8383
(defn- refresh-reply
@@ -86,7 +86,7 @@
8686
(exec id
8787
(fn []
8888
(locking refresh-tracker
89-
(when @user-indicated-clear?
89+
(when @client-requested-clear?
9090
(vreset! refresh-tracker (track/tracker)))
9191
(vswap! refresh-tracker
9292
(fn [tracker]
@@ -107,7 +107,7 @@
107107
tracker)
108108

109109
(finally
110-
(reset! user-indicated-clear? false)))))))
110+
(reset! client-requested-clear? false)))))))
111111
(fn []
112112
(transport/send transport (response-for msg {:status :done}))))))
113113

@@ -116,10 +116,10 @@
116116
(let [{:keys [exec]} (meta session)]
117117
(exec id
118118
(fn []
119-
;; This used to be a `locking`-based call to our `clear` implementation
120-
;; Now it merely "enqueues" a clearing, because the "refresh-clear" op is called synchronously by CIDER,
121-
;; so the `locking` could cause timeouts (https://github.com/clojure-emacs/cider/issues/3652 )
122-
(reset! user-indicated-clear? true))
119+
;; This used to be a `locking`-based call to our `clear` implementation.
120+
;; Now it merely "enqueues" a clearing,
121+
;; because that `locking` could cause unnecessary nREPL timeouts (https://github.com/clojure-emacs/cider/issues/3652 ).
122+
(reset! client-requested-clear? true))
123123
(fn []
124124
(transport/send transport (response-for msg {:status :done}))))))
125125

0 commit comments

Comments
 (0)