Re: [PATCH 1/2] emacs: add defsubst notmuch-address--message-insinuated

2015-08-30 Thread Tomi Ollila
On Sun, Aug 30 2015, David Bremner wrote: > Tomi Ollila writes: > >> +(defsubst notmuch-address--message-insinuated () >> + (memq notmuch-address-message-alist-member message-completion-alist)) >> + > > Is there some advantage to defsubst other than (maybe?) performance? It > just seems like o

Re: [PATCH 1/2] emacs: add defsubst notmuch-address--message-insinuated

2015-08-30 Thread David Bremner
Tomi Ollila writes: > +(defsubst notmuch-address--message-insinuated () > + (memq notmuch-address-message-alist-member message-completion-alist)) > + Is there some advantage to defsubst other than (maybe?) performance? It just seems like one more construct for people to get up to speed with th

[PATCH 1/2] emacs: add defsubst notmuch-address--message-insinuated

2015-08-26 Thread Tomi Ollila
This "inline function" is currently used in notmuch-address-message-insinuate (to not enable address completion if it is already enabled). In some functions later this will be called to know whether address completion can be used there, too. --- Previous version of this 2-patch series is at id:142