[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[nongnu] elpa/inf-clojure f7ec13a 160/313: Improve performance of inf-cl
From: |
ELPA Syncer |
Subject: |
[nongnu] elpa/inf-clojure f7ec13a 160/313: Improve performance of inf-clojure--process-response and close #89 |
Date: |
Wed, 11 Aug 2021 10:00:07 -0400 (EDT) |
branch: elpa/inf-clojure
commit f7ec13ab2fbc36e99df196eb863a75baf3b92fd6
Author: Andrea Richiardi <a.richiardi.work@gmail.com>
Commit: Bozhidar Batsov <bozhidar.batsov@gmail.com>
Improve performance of inf-clojure--process-response and close #89
Introduce double checking in the busy wait that detects if the inf-clojure
process (the REPL) has completed and sent back everything. This removes the
slowness that was triggered by having accept-process-output in the while
body:
we can take advantage now of its output to wait as well.
---
CHANGELOG.md | 1 +
inf-clojure.el | 5 +++--
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/CHANGELOG.md b/CHANGELOG.md
index 4b17de3..6421134 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -6,6 +6,7 @@
* [#79](https://github.com/clojure-emacs/inf-clojure/pull/82): Eldoc error
when running boot repl.
* [#83](https://github.com/clojure-emacs/inf-clojure/pull/85): No such
namespace: complete.core in lumo REPL.
+* [#93](https://github.com/clojure-emacs/inf-clojure/pull/93): Slow response
from inf-clojure (completions, arglists, ...).
## 2.0.1 (2017-05-18)
diff --git a/inf-clojure.el b/inf-clojure.el
index 95cc6da..a803d1e 100644
--- a/inf-clojure.el
+++ b/inf-clojure.el
@@ -996,8 +996,9 @@ the `inf-clojure-prompt`."
(inf-clojure--sanitize-command command) work-buffer process nil t)
;; Wait for the process to complete
(set-buffer (process-buffer process))
- (while (null comint-redirect-completed)
- (accept-process-output nil 1))
+ (while (and (null comint-redirect-completed)
+ (accept-process-output process 1 0 t))
+ (sleep-for 0.01))
;; Collect the output
(set-buffer work-buffer)
(goto-char (point-min))
- [nongnu] elpa/inf-clojure 2669544 129/313: [Fix #63] Avoid spurious output by using buffer redirection, (continued)
- [nongnu] elpa/inf-clojure 2669544 129/313: [Fix #63] Avoid spurious output by using buffer redirection, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 0fbcfcb 140/313: Improve the README a bit, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure db7bfaf 155/313: Fix the changelog, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 42f860f 169/313: Add Multiple Process Support section to README, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 3a93815 170/313: [Fix #105] Add planck-specific function for show-arglist (#106), ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 204e5b7 114/313: Do some cleanup in preparation for a major release, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure d81e266 116/313: [Fix #48] Stop prompting for the symbol at point by default (#51), ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 431c068 127/313: Fix three "variable is void" bugs, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 723064d 151/313: [Fix #83] Harden parsing of repl responses (#85), ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure a705085 152/313: Add nil parameter to lumo completions' atom, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure f7ec13a 160/313: Improve performance of inf-clojure--process-response and close #89,
ELPA Syncer <=
- [nongnu] elpa/inf-clojure 12583f4 175/313: Add :safe clause for inf-clojure-repl-use-same-window, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 247ca70 182/313: Release 2.1.0, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 62bb0fc 186/313: Update CHANGELOG.md, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure dcb523b 191/313: Avoid computing completion bounds when no valid chars are at point, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 1507faa 119/313: Add dir support in Lumo, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure ff72d66 236/313: [Fix #171] Release 2.2.0, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure ac40726 241/313: Move all repl stuff into single nested alist, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 24121e5 246/313: Add babashka repl and readme cleanups, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 977e176 258/313: Mark this package as not part of Emacs, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 4af94e6 271/313: Fixed two checkdoc warnings, ELPA Syncer, 2021/08/11