Re: [BUG] Notmuch keeps adding notmuch-address-expand-name to message-completion-alist

2017-09-11 Thread Attic Hermit
You can get rid of the bug by using `member` instead of `memq` in
`nomtuch-address-setup`:

diff -u a/notmuch-address.el b/notmuch-address.el
--- a/notmuch-address.el2017-08-05 18:45:40.0 +0900
+++ b/notmuch-address.el2017-09-11 19:40:41.230872915 +0900
@@ -159,7 +159,7 @@
   #'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)

David Bremner  writes:

> Attic Hermit  writes:
>
>> When I invocate compose-mail or something related with the message-mode
>> buffer and Notmuch, Notmuch keeps adding a pair like below to
>> `message-completion-alist`:
>>
>>   
>> ("^\\(Resent-\\)?\\(To\\|B?Cc\\|Reply-To\\|From\\|Mail-Followup-To\\|Mail-Copies-To\\):"
>>   . notmuch-address-expand-name)
>>
>
> I don't know much about the cause, but I can confirm that many such
> pairs are added to this list.
>
> d
-- 
Attic Hermit
___
notmuch mailing list
notmuch@notmuchmail.org
https://notmuchmail.org/mailman/listinfo/notmuch


Re: [BUG] Notmuch keeps adding notmuch-address-expand-name to message-completion-alist

2017-09-11 Thread David Bremner
Attic Hermit  writes:

> When I invocate compose-mail or something related with the message-mode
> buffer and Notmuch, Notmuch keeps adding a pair like below to
> `message-completion-alist`:
>
>   
> ("^\\(Resent-\\)?\\(To\\|B?Cc\\|Reply-To\\|From\\|Mail-Followup-To\\|Mail-Copies-To\\):"
>   . notmuch-address-expand-name)
>

I don't know much about the cause, but I can confirm that many such
pairs are added to this list.

d
___
notmuch mailing list
notmuch@notmuchmail.org
https://notmuchmail.org/mailman/listinfo/notmuch


[BUG] Notmuch keeps adding notmuch-address-expand-name to message-completion-alist

2017-09-10 Thread Attic Hermit
When I invocate compose-mail or something related with the message-mode
buffer and Notmuch, Notmuch keeps adding a pair like below to
`message-completion-alist`:

  
("^\\(Resent-\\)?\\(To\\|B?Cc\\|Reply-To\\|From\\|Mail-Followup-To\\|Mail-Copies-To\\):"
  . notmuch-address-expand-name)

I think there's bug in `notmuch-address-setup` on `notmuch-address.el`.
It checks that the pair exists in the `message-completion-alist` by
`memq`, but it seems to be amended to `member` instead.

P.S. Because I'm not a programming expert or a student, I don't know how
to send a patch to amend this problem. So I send where the bug was found
and the way how I tinkered it in English. I searched the file where bug
exists in the git repository of Notmuch, and found that there still was
a bug. I hope this will help to fix the bug.
-- 
Attic Hermit
___
notmuch mailing list
notmuch@notmuchmail.org
https://notmuchmail.org/mailman/listinfo/notmuch