diff options
| author | Jose Antonio Ortega Ruiz <jao@gnu.org> | 2010-11-26 23:05:34 +0100 |
|---|---|---|
| committer | Jose Antonio Ortega Ruiz <jao@gnu.org> | 2010-11-26 23:05:34 +0100 |
| commit | 9aedb7dd472ca9d842ea2a6b9c412c66e3d2b849 (patch) | |
| tree | 900711aa614238c7a7a01646655b0351b341fc40 /elisp | |
| parent | b30cc842ed1a0cab0b6e68375133820478ba12bd (diff) | |
| download | geiser-9aedb7dd472ca9d842ea2a6b9c412c66e3d2b849.tar.gz geiser-9aedb7dd472ca9d842ea2a6b9c412c66e3d2b849.tar.bz2 | |
Autodoc not interning symbols
We avoid calling symbol-at-point, and keep the cached signatures with
strings as keys.
Diffstat (limited to 'elisp')
| -rw-r--r-- | elisp/geiser-autodoc.el | 19 | ||||
| -rw-r--r-- | elisp/geiser-syntax.el | 14 |
2 files changed, 20 insertions, 13 deletions
diff --git a/elisp/geiser-autodoc.el b/elisp/geiser-autodoc.el index 5612296..7c61b33 100644 --- a/elisp/geiser-autodoc.el +++ b/elisp/geiser-autodoc.el @@ -65,19 +65,22 @@ when `geiser-autodoc-display-module-p' is on." (if (not geiser-autodoc--cached-signatures) (setq missing funs) (dolist (f funs) - (let ((cf (assq f geiser-autodoc--cached-signatures))) + (let ((cf (assoc f geiser-autodoc--cached-signatures))) (if cf (push cf cached) (push f missing))))) (unless (or cached keep-cached) (geiser-autodoc--clean-cache)) (when missing - (let ((res (geiser-eval--send/result `(:eval (:ge autodoc - (quote ,missing))) - 500))) + (let* ((missing (mapcar 'make-symbol missing)) + (res (geiser-eval--send/result + `(:eval (:ge autodoc (quote ,missing))) 500))) (when res (setq geiser-autodoc--cached-signatures - (append res (if keep-cached - geiser-autodoc--cached-signatures - cached)))))))) + (append (mapcar (lambda (s) + (cons (format "%s" (car s)) (cdr s))) + res) + (if keep-cached + geiser-autodoc--cached-signatures + cached)))))))) geiser-autodoc--cached-signatures))) (defun geiser-autodoc--sanitize-args (args) @@ -176,7 +179,7 @@ when `geiser-autodoc-display-module-p' is on." (p (car path)) (s)) (while (and p (not s)) - (unless (setq s (cdr (assq (car p) signs))) + (unless (setq s (cdr (assoc (car p) signs))) (setq p (car path)) (setq path (cdr path)))) (when s (geiser-autodoc--str p s)))) diff --git a/elisp/geiser-syntax.el b/elisp/geiser-syntax.el index 2bfc1c7..ae1c3dd 100644 --- a/elisp/geiser-syntax.el +++ b/elisp/geiser-syntax.el @@ -232,8 +232,10 @@ implementation-specific entries for font-lock-keywords.") (defsubst geiser-syntax--symbol-at-point () (and (not (nth 8 (syntax-ppss))) - (let ((s (symbol-at-point))) - (and (not (eq s '.)) s)))) + (let ((s (thing-at-point 'symbol))) + (and s + (not (equal s ".")) + (make-symbol (substring-no-properties s)))))) (defsubst geiser-syntax--skip-comment/string () (let ((pos (nth 8 (syntax-ppss)))) @@ -256,8 +258,9 @@ implementation-specific entries for font-lock-keywords.") (skip-syntax-forward "-<>") (when (<= (point) boundary) (forward-sexp) - (let ((s (symbol-at-point))) - (when (not (eq s '.)) (push s elems)))))) + (let ((s (thing-at-point 'symbol))) + (cond ((not s) (push s elems)) + ((not (equal "." s)) (push (make-symbol s) elems))))))) (nreverse elems))))) (defun geiser-syntax--scan-sexps (&optional begin) @@ -276,7 +279,8 @@ implementation-specific entries for font-lock-keywords.") (prev (and (> pos 1) (nth (1- pos) form))) (prev (and (keywordp prev) (list prev)))) (push `(,(car form) ,pos ,@prev) path))))))) - (nreverse path))) + (mapcar (lambda (e) (cons (format "%s" (car e)) (cdr e))) + (nreverse path)))) (defsubst geiser-syntax--binding-form-p (bfs sbfs f) (or (memq f '(define define* define-syntax define-syntax-rule |