[PATCH v2] emacs: When completing tags, offer each tag once

[PATCH v2] emacs: When completing tags, offer each tag once

[David Edmondson]

When prompting for one or more tags to add or remove to/from one or
more threads, ensure that the set of tags offered for completion
contains no duplicates.

Some completion packages (e.g. selectrum) will include every member of
the offered list, resulting in the same tag being indicated as a
possibility several times.
---
Use `delete-dups' rather than hand-coding similar.

 emacs/notmuch.el | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/emacs/notmuch.el b/emacs/notmuch.el
index 26efcccd..6d37c623 100644
--- a/emacs/notmuch.el
+++ b/emacs/notmuch.el
@@ -577,7 +577,7 @@ thread."
     (notmuch-search-foreach-result beg end
       (lambda (pos)
 	(setq output (append output (notmuch-search-get-tags pos)))))
-    output))
+    (delete-dups output)))
 
 (defun notmuch-search-interactive-tag-changes (&optional initial-input)
   "Prompt for tag changes for the current thread or region.
-- 
2.30.0

Re: [PATCH v2] emacs: When completing tags, offer each tag once

[David Bremner]

David Edmondson <dme@dme.org> writes:

> When prompting for one or more tags to add or remove to/from one or
> more threads, ensure that the set of tags offered for completion
> contains no duplicates.

applied to master

d