X-Git-Url: https://code.delx.au/gnu-emacs-elpa/blobdiff_plain/2e6b4f66cf6bc14868f23a350063922de3b37d0c..41ef088456675919e4b56a41f964d50a81a781dc:/packages/company/company-capf.el diff --git a/packages/company/company-capf.el b/packages/company/company-capf.el index 1a4342bf5..e27ae7a46 100644 --- a/packages/company/company-capf.el +++ b/packages/company/company-capf.el @@ -25,20 +25,23 @@ ;;; Code: -(eval-when-compile (require 'cl)) +(require 'company) +(require 'cl-lib) (defun company--capf-data () - ;; Ignore tags-completion-at-point-function because it subverts company-etags - ;; in the default value of company-backends, where the latter comes later. - (letf* (((default-value 'completion-at-point-functions) nil) - (data (run-hook-wrapped 'completion-at-point-functions - ;; Ignore misbehaving functions. - #'completion--capf-wrapper 'optimist))) - (when (and (consp data) (numberp (nth 1 data))) data))) + (cl-letf* (((default-value 'completion-at-point-functions) + ;; Ignore tags-completion-at-point-function because it subverts + ;; company-etags in the default value of company-backends, where + ;; the latter comes later. + (remove 'tags-completion-at-point-function + (default-value 'completion-at-point-functions))) + (data (run-hook-wrapped 'completion-at-point-functions + ;; Ignore misbehaving functions. + #'completion--capf-wrapper 'optimist))) + (when (and (consp (cdr data)) (numberp (nth 1 data))) data))) (defun company-capf (command &optional arg &rest _args) - "`company-mode' back-end using `completion-at-point-functions'. -Requires Emacs 24.1 or newer." + "`company-mode' back-end using `completion-at-point-functions'." (interactive (list 'interactive)) (pcase command (`interactive (company-begin-backend 'company-capf)) @@ -77,18 +80,18 @@ Requires Emacs 24.1 or newer." (buffer-substring (nth 1 res) (nth 2 res)) (nth 3 res) (plist-get (nthcdr 4 res) :predicate)))) (cdr (assq 'display-sort-function meta)))))) - (`common-part + (`match ;; Can't just use 0 when base-size (see above) is non-zero. - (let ((start (if (get-text-property 0 'face arg) + (let ((start (if (get-text-property 0 'font-lock-face arg) 0 - (next-single-property-change 0 'face arg)))) + (next-single-property-change 0 'font-lock-face arg)))) (when start ;; completions-common-part comes first, but we can't just look for this ;; value because it can be in a list. (or - (let ((value (get-text-property start 'face arg))) + (let ((value (get-text-property start 'font-lock-face arg))) (text-property-not-all start (length arg) - 'face value arg)) + 'font-lock-face value arg)) (length arg))))) (`duplicates t) (`no-cache t) ;Not much can be done here, as long as we handle @@ -104,7 +107,17 @@ Requires Emacs 24.1 or newer." (when f (funcall f arg)))) (`annotation (save-excursion - (goto-char company-point) + ;; 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. 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))) (when f (funcall f arg))))) (`require-match