[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[elpa] 10/16: company-capf: get rid of caching last capf-data
From: |
Dmitry Gutov |
Subject: |
[elpa] 10/16: company-capf: get rid of caching last capf-data |
Date: |
Sat, 26 Jul 2014 04:53:22 +0000 |
dgutov pushed a commit to branch master
in repository elpa.
commit 3f14abc2a5d72bdc9836b34ddb5e0ab2475a0bd4
Author: Dmitry Gutov <address@hidden>
Date: Sun Jul 20 09:14:06 2014 +0300
company-capf: get rid of caching last capf-data
It didn't work well enough.
Fixes #153.
---
company-capf.el | 34 +++++++++++++++-------------------
1 files changed, 15 insertions(+), 19 deletions(-)
diff --git a/company-capf.el b/company-capf.el
index cc075df..e27ae7a 100644
--- a/company-capf.el
+++ b/company-capf.el
@@ -28,13 +28,6 @@
(require 'company)
(require 'cl-lib)
-(defvar-local company--capf-data nil)
-
-(defun company--capf-clear-data (&optional _ignore)
- (setq company--capf-data nil)
- (remove-hook 'company-completion-cancelled-hook 'company--capf-clear-data t)
- (remove-hook 'company-completion-finished-hook 'company--capf-clear-data t))
-
(defun company--capf-data ()
(cl-letf* (((default-value 'completion-at-point-functions)
;; Ignore tags-completion-at-point-function because it subverts
@@ -57,12 +50,9 @@
(when res
(if (> (nth 2 res) (point))
'stop
- (setq company--capf-data res)
- (add-hook 'company-completion-cancelled-hook
'company--capf-clear-data nil t)
- (add-hook 'company-completion-finished-hook
'company--capf-clear-data nil t)
(buffer-substring-no-properties (nth 1 res) (point))))))
(`candidates
- (let ((res company--capf-data))
+ (let ((res (company--capf-data)))
(when res
(let* ((table (nth 3 res))
(pred (plist-get (nthcdr 4 res) :predicate))
@@ -84,7 +74,7 @@
candidates))
candidates)))))
(`sorted
- (let ((res company--capf-data))
+ (let ((res (company--capf-data)))
(when res
(let ((meta (completion-metadata
(buffer-substring (nth 1 res) (nth 2 res))
@@ -107,28 +97,34 @@
(`no-cache t) ;Not much can be done here, as long as we handle
;non-prefix matches.
(`meta
- (let ((f (plist-get (nthcdr 4 company--capf-data) :company-docsig)))
+ (let ((f (plist-get (nthcdr 4 (company--capf-data)) :company-docsig)))
(when f (funcall f arg))))
(`doc-buffer
- (let ((f (plist-get (nthcdr 4 company--capf-data) :company-doc-buffer)))
+ (let ((f (plist-get (nthcdr 4 (company--capf-data)) :company-doc-buffer)))
(when f (funcall f arg))))
(`location
- (let ((f (plist-get (nthcdr 4 company--capf-data) :company-location)))
+ (let ((f (plist-get (nthcdr 4 (company--capf-data)) :company-location)))
(when f (funcall f arg))))
(`annotation
(save-excursion
;; FIXME: `company-begin' sets `company-point' after calling
;; `company--begin-new'. We shouldn't rely on `company-point' here,
- ;; better to cache the capf-data value instead.
+ ;; better to cache the capf-data value instead. However: we can't just
+ ;; save the last capf-data value in `prefix', because that command can
+ ;; get called more often than `candidates', and at any point in the
+ ;; buffer (https://github.com/company-mode/company-mode/issues/153).
+ ;; We could try propertizing the returned prefix string, but it's not
+ ;; passed to `annotation', and `company-prefix' is set only after
+ ;; `company--strip-duplicates' is called.
(when company-point
(goto-char company-point))
- (let ((f (plist-get (nthcdr 4 company--capf-data)
:annotation-function)))
+ (let ((f (plist-get (nthcdr 4 (company--capf-data))
:annotation-function)))
(when f (funcall f arg)))))
(`require-match
- (plist-get (nthcdr 4 company--capf-data) :company-require-match))
+ (plist-get (nthcdr 4 (company--capf-data)) :company-require-match))
(`init nil) ;Don't bother: plenty of other ways to initialize the
code.
(`post-completion
- (let* ((res company--capf-data)
+ (let* ((res (company--capf-data))
(exit-function (plist-get (nthcdr 4 res) :exit-function)))
(if exit-function
(funcall exit-function arg 'finished))))
- [elpa] branch master updated (68ccff3 -> ca001a5), Dmitry Gutov, 2014/07/26
- [elpa] 03/16: company-pseudo-tooltip-frontend: Small rewrite, Dmitry Gutov, 2014/07/26
- [elpa] 06/16: +1 char, -1 line, Dmitry Gutov, 2014/07/26
- [elpa] 05/16: company-pseudo-tooltip-guard: Stabilize when the popup is displayed up, Dmitry Gutov, 2014/07/26
- [elpa] 09/16: company-sort-by-occurrence: continue search after predicate fails, Dmitry Gutov, 2014/07/26
- [elpa] 07/16: Deprecate setting company-idle-delay to t, Dmitry Gutov, 2014/07/26
- [elpa] 08/16: company-sort-by-occurrence: allow customizing the weighing logic, Dmitry Gutov, 2014/07/26
- [elpa] 11/16: company-css-property-value-regexp: prohibit opening curlies, Dmitry Gutov, 2014/07/26
- [elpa] 13/16: Fix company-occurrence-weight-function, Dmitry Gutov, 2014/07/26
- [elpa] 12/16: company-css-property-values: delete duplicates, Dmitry Gutov, 2014/07/26
- [elpa] 10/16: company-capf: get rid of caching last capf-data,
Dmitry Gutov <=
- [elpa] 14/16: Fix tests in batch mode, Dmitry Gutov, 2014/07/26
- [elpa] 01/16: company-frontends-set: never modify VALUE destructively, Dmitry Gutov, 2014/07/26
- [elpa] 15/16: Release 0.8.2, Dmitry Gutov, 2014/07/26
- [elpa] 16/16: Merge commit 'b1d019a4c815ac8bdc240d69eaa74eb4e34640e8' from company-master, Dmitry Gutov, 2014/07/26
- [elpa] 04/16: company-pseudo-tooltip-guard: Invalidate after crossing line continuation, Dmitry Gutov, 2014/07/26
- [elpa] 02/16: Fix popup when prefix is split by line continuation, Dmitry Gutov, 2014/07/26