[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[nongnu] elpa/inf-clojure 7441d3b 120/313: Fix REPL type detection
From: |
ELPA Syncer |
Subject: |
[nongnu] elpa/inf-clojure 7441d3b 120/313: Fix REPL type detection |
Date: |
Wed, 11 Aug 2021 09:59:59 -0400 (EDT) |
branch: elpa/inf-clojure
commit 7441d3b0d2255241916e33dcb19da6a206202fa3
Author: Andrea Richiardi <a.richiardi.work@gmail.com>
Commit: Bozhidar Batsov <bozhidar.batsov@gmail.com>
Fix REPL type detection
The problem was occurring when the first operation was not an
`inf-clojure--send-string`. Now `inf-clojure--set-repl-type` is
explicitely called in the form getters.
---
inf-clojure.el | 12 +++++++-----
1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/inf-clojure.el b/inf-clojure.el
index e47095a..fe6aa7d 100644
--- a/inf-clojure.el
+++ b/inf-clojure.el
@@ -202,9 +202,11 @@ either `setq-local` or an entry in `.dir-locals.el`." )
(t 'clojure)))
(defun inf-clojure--set-repl-type (proc)
- "Set the REPL type if has not already been set."
- (when (not inf-clojure-repl-type)
- (setq inf-clojure-repl-type (inf-clojure--detect-type proc))))
+ "Set the REPL type if has not already been set.
+It requires a REPL PROC for inspecting the correct type."
+ (if (not inf-clojure-repl-type)
+ (setq inf-clojure-repl-type (inf-clojure--detect-type proc))
+ inf-clojure-repl-type))
(defun inf-clojure--send-string (proc string)
"A custom `comint-input-sender` / `comint-send-string`.
@@ -584,7 +586,7 @@ The prefix argument SWITCH-TO-REPL controls whether to
switch to REPL after the
"Return the form to query inferior Clojure for a var's documentation.
If you are using REPL types, it will pickup the most approapriate
`inf-clojure-var-doc-form` variant."
- (pcase inf-clojure-repl-type
+ (pcase (inf-clojure--set-repl-type (inf-clojure-proc))
(`lumo inf-clojure-var-doc-form-lumo)
(_ inf-clojure-var-doc-form)))
@@ -641,7 +643,7 @@ If you are using REPL types, it will pickup the most
approapriate
"Return the form to query inferior Clojure for a var's documentation.
If you are using REPL types, it will pickup the most approapriate
`inf-clojure-completion-form` variant."
- (pcase inf-clojure-repl-type
+ (pcase (inf-clojure--set-repl-type (inf-clojure-proc))
(`lumo inf-clojure-completion-form-lumo)
(_ inf-clojure-completion-form)))
- [nongnu] elpa/inf-clojure e291da3 274/313: Update README.md, (continued)
- [nongnu] elpa/inf-clojure e291da3 274/313: Update README.md, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 787400f 264/313: [Fix #178] Ensure there is a directory for comint (thanks mcphailm), ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 6cb8e43 277/313: Update the CI images, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 477fd7c 272/313: Tweak some wording, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure a98a260 281/313: [#152] Add a changelog entry, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure f4a279e 286/313: [Fix #177] Don't run the tests on Emacs 25, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure b4193fc 289/313: Reindent tests, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure f24722c 294/313: Extract useful functions, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure e1a93c7 121/313: Add lock around REPL type detection, ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 5ef6bdf 123/313: Revert "Remove unwanted ansi chars from the REPL output", ELPA Syncer, 2021/08/11
- [nongnu] elpa/inf-clojure 7441d3b 120/313: Fix REPL type detection,
ELPA Syncer <=
- [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