[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[nongnu] elpa/inf-clojure 6c719c6 197/313: Avoid leaking buffers from in
From: |
ELPA Syncer |
Subject: |
[nongnu] elpa/inf-clojure 6c719c6 197/313: Avoid leaking buffers from inf-clojure--process-response |
Date: |
Wed, 11 Aug 2021 10:00:15 -0400 (EDT) |
branch: elpa/inf-clojure
commit 6c719c6e8bdaedf4f98888bcba31afe1989c9229
Author: Andrea Richiardi <a.richiardi.work@gmail.com>
Commit: Bozhidar Batsov <bozhidar.batsov@gmail.com>
Avoid leaking buffers from inf-clojure--process-response
---
inf-clojure.el | 34 +++++++++++++++++-----------------
1 file changed, 17 insertions(+), 17 deletions(-)
diff --git a/inf-clojure.el b/inf-clojure.el
index e9920f9..817147d 100644
--- a/inf-clojure.el
+++ b/inf-clojure.el
@@ -1220,25 +1220,25 @@ output from and including the `inf-clojure-prompt`."
(sanitized-command (inf-clojure--sanitize-command command)))
(when (not (string-empty-p sanitized-command))
(inf-clojure--log-string command "----CMD->")
- (set-buffer (inf-clojure--get-redirect-buffer))
- (erase-buffer)
- (comint-redirect-send-command-to-process sanitized-command
redirect-buffer-name process nil t)
+ (with-current-buffer (inf-clojure--get-redirect-buffer)
+ (erase-buffer)
+ (comint-redirect-send-command-to-process sanitized-command
redirect-buffer-name process nil t))
;; Wait for the process to complete
- (set-buffer (process-buffer process))
- (while (and (null comint-redirect-completed)
- (accept-process-output process 1 0 t))
- (sleep-for 0.01))
+ (with-current-buffer (process-buffer process)
+ (while (and (null comint-redirect-completed)
+ (accept-process-output process 1 0 t))
+ (sleep-for 0.01)))
;; Collect the output
- (set-buffer redirect-buffer-name)
- (goto-char (point-min))
- (let* ((buffer-string (buffer-substring-no-properties (point-min)
(point-max)))
- (boundaries (inf-clojure--string-boundaries buffer-string
inf-clojure-prompt beg-regexp end-regexp))
- (beg-pos (car boundaries))
- (end-pos (car (cdr boundaries)))
- (prompt-pos (car (cdr (cdr boundaries))))
- (response-string (substring buffer-string beg-pos (min end-pos
prompt-pos))))
- (inf-clojure--log-string buffer-string "<-RES----")
- response-string))))
+ (with-current-buffer redirect-buffer-name
+ (goto-char (point-min))
+ (let* ((buffer-string (buffer-substring-no-properties (point-min)
(point-max)))
+ (boundaries (inf-clojure--string-boundaries buffer-string
inf-clojure-prompt beg-regexp end-regexp))
+ (beg-pos (car boundaries))
+ (end-pos (car (cdr boundaries)))
+ (prompt-pos (car (cdr (cdr boundaries))))
+ (response-string (substring buffer-string beg-pos (min end-pos
prompt-pos))))
+ (inf-clojure--log-string buffer-string "<-RES----")
+ response-string)))))
(defun inf-clojure--nil-string-match-p (string)
"Return true iff STRING is not nil.
- [nongnu] elpa/inf-clojure 864356a 172/313: [Fix #75] Remove compilation warnings (#109), (continued)
- [nongnu] elpa/inf-clojure 864356a 172/313: [Fix #75] Remove compilation warnings (#109), ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 5d76920 176/313: Introduce inf-clojure-project-type defcustom, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure f4478ad 183/313: Fix and reword docstring, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure d4010db 190/313: Add @ (deref) and ^ to the expression breaking chars, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 37b30be 157/313: Bind C-c M-c to inf-clojure-connect, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 49b8c4b 159/313: Restyle inf-clojure-log-activity output, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 21d3e97 171/313: Bump the development version to 2.1.0-snapshot, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 39e794c 168/313: Protect against empty commands, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure ae43e2e 178/313: [Fix #113] Send REPL string always, even if empty, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure c4adc89 181/313: Introduce inf-clojure-completions-fn defcustom, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 6c719c6 197/313: Avoid leaking buffers from inf-clojure--process-response,
ELPA Syncer <=
- [nongnu] elpa/inf-clojure 5aa9c15 212/313: Add clojure socket REPL command and tweak, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure b87c5fb 215/313: update instructions to setup lumo repl (#147), ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 16455ab 224/313: Fix typo in inf-clojure--joker-repl-form (#163), ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 975e580 222/313: Fix CircleCI badge in README, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 173d0e7 228/313: Bump the development version, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 80cefbb 244/313: API to update repl-features, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 3e7ca33 248/313: Add lein and boot to the startup-forms, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 1c8429a 247/313: Bump the development version, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 5abac0b 262/313: Tweak the REPL type documentation, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 6f4e792 065/313: Add a note about mixing cider & inf-clojure, ELPA Syncer, 2021/08/11