[PATCH] emacs/address: use member instead of memq to check for pair

David Edmondson dme at dme.org
Wed Sep 13 00:55:31 PDT 2017


Looks good.

On Tuesday, 2017-09-12 at 22:31:46 -0300, David Bremner wrote:

> Quoting from the elisp reference:
>
>      For other types (e.g., lists, vectors, strings), two arguments
>      with the same contents or elements are not necessarily ‘eq’ to
>      each other.
>
> Thanks to "Attic Hermit" for the fix.
> ---
>  emacs/notmuch-address.el | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/emacs/notmuch-address.el b/emacs/notmuch-address.el
> index f937e708..64887a43 100644
> --- a/emacs/notmuch-address.el
> +++ b/emacs/notmuch-address.el
> @@ -159,7 +159,7 @@ matching `notmuch-address-completion-headers-regexp'.
>  		       #'notmuch-address-expand-name)))
>        (when setup-company
>  	(notmuch-company-setup))
> -      (unless (memq pair message-completion-alist)
> +      (unless (member pair message-completion-alist)
>  	(setq message-completion-alist
>  	      (push pair message-completion-alist)))))
>  
> -- 
> 2.14.1
>
> _______________________________________________
> notmuch mailing list
> notmuch at notmuchmail.org
> https://notmuchmail.org/mailman/listinfo/notmuch

dme.
-- 
And the sign said: long haired freaky people need not apply.


More information about the notmuch mailing list