[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
scratch/bug-42149-funny-pcm-completion-scores d8c596f 3/4: Robustify a h
From: |
João Távora |
Subject: |
scratch/bug-42149-funny-pcm-completion-scores d8c596f 3/4: Robustify a helper function for test/lisp/minibuffer-tests.el |
Date: |
Tue, 29 Dec 2020 08:32:33 -0500 (EST) |
branch: scratch/bug-42149-funny-pcm-completion-scores
commit d8c596f7309bd6fd6e127b8027dfb4c508afd2ea
Author: João Távora <joaotavora@gmail.com>
Commit: João Távora <joaotavora@gmail.com>
Robustify a helper function for test/lisp/minibuffer-tests.el
completion--pcm-first-difference-pos wasn't taking into account the
fact that faces may come in lists. bug#42149
* test/lisp/minibuffer-tests.el
(completion--pcm-first-difference-pos): Robustify.
---
test/lisp/minibuffer-tests.el | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/test/lisp/minibuffer-tests.el b/test/lisp/minibuffer-tests.el
index d665587b..381096f 100644
--- a/test/lisp/minibuffer-tests.el
+++ b/test/lisp/minibuffer-tests.el
@@ -112,11 +112,14 @@
(get-text-property 0 'completion-score comp))
(defun completion--pcm-first-difference-pos (comp)
- (cl-loop for pos = (next-single-property-change 0 'face comp)
+ (cl-loop with faces
+ for pos = (next-single-property-change 0 'face comp)
then (next-single-property-change pos 'face comp)
while pos
- when (eq (get-text-property pos 'face comp)
- 'completions-first-difference)
+ when (or (eq (setq faces (get-text-property pos 'face comp))
+ 'completions-first-difference)
+ (and (listp faces)
+ (memq 'completions-first-difference faces)))
return pos))
(ert-deftest completion-pcm-test-1 ()