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

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

Date: Sun, 30 Aug 2015 09:28:32 -0300

To: Tomi Ollila, notmuch@notmuchmail.org

Cc: tomi.ollila@iki.fi

From: David Bremner

Tomi Ollila <tomi.ollila@iki.fi> 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
the codebase.

d

• Previous message (by thread): Re: [PATCH 2/2] emacs: add function to resend message to new recipients

Thread:

• Tomi Ollila—[PATCH 1/2] emacs: add defsubst notmuch-address--message-insinuated [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  • Tomi Ollila—[PATCH 2/2] emacs: add function to resend message to new recipients [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
    • David Bremner—Re: [PATCH 2/2] emacs: add function to resend message to new recipients [inbox, unread]
      • Tomi Ollila—Re: [PATCH 2/2] emacs: add function to resend message to new recipients [inbox, unread]
        • David Bremner—Re: [PATCH 2/2] emacs: add function to resend message to new recipients [inbox, unread]
          • Tomi Ollila—Re: [PATCH 2/2] emacs: add function to resend message to new recipients [inbox, unread]
            • David Bremner—Re: [PATCH 2/2] emacs: add function to resend message to new recipients [inbox, unread]
  • David Bremner—Re: [PATCH 1/2] emacs: add defsubst notmuch-address--message-insinuated [inbox, unread]
    • Tomi Ollila—Re: [PATCH 1/2] emacs: add defsubst notmuch-address--message-insinuated [inbox, unread]