5b/ed18b3c8aa46522d13856896ca629f49fa992e

   1 Return-Path: <david@tethera.net>\r
   2 X-Original-To: notmuch@notmuchmail.org\r
   3 Delivered-To: notmuch@notmuchmail.org\r
   4 Received: from localhost (localhost [127.0.0.1])\r
   5  by arlo.cworth.org (Postfix) with ESMTP id 9FA006DE17FD\r
   6  for <notmuch@notmuchmail.org>; Sun, 30 Aug 2015 05:29:52 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at cworth.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: 0.117\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0.117 tagged_above=-999 required=5 tests=[AWL=0.117]\r
  12  autolearn=disabled\r
  13 Received: from arlo.cworth.org ([127.0.0.1])\r
  14  by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15  with ESMTP id uEMaZQ06zLBS for <notmuch@notmuchmail.org>;\r
  16  Sun, 30 Aug 2015 05:29:50 -0700 (PDT)\r
  17 Received: from gitolite.debian.net (gitolite.debian.net [87.98.215.224])\r
  18  by arlo.cworth.org (Postfix) with ESMTPS id 9CBB86DE17EB\r
  19  for <notmuch@notmuchmail.org>; Sun, 30 Aug 2015 05:29:50 -0700 (PDT)\r
  20 Received: from remotemail by gitolite.debian.net with local (Exim 4.80)\r
  21  (envelope-from <david@tethera.net>)\r
  22  id 1ZW1jB-0007LW-Co; Sun, 30 Aug 2015 12:28:53 +0000\r
  23 Received: (nullmailer pid 3529 invoked by uid 1000); Sun, 30 Aug 2015\r
  24  12:28:32 -0000\r
  25 From: David Bremner <david@tethera.net>\r
  26 To: Tomi Ollila <tomi.ollila@iki.fi>, notmuch@notmuchmail.org\r
  27 Subject: Re: [PATCH 1/2] emacs: add defsubst\r
  28  notmuch-address--message-insinuated\r
  29 In-Reply-To: <1440619626-18768-1-git-send-email-tomi.ollila@iki.fi>\r
  30 References: <1440619626-18768-1-git-send-email-tomi.ollila@iki.fi>\r
  31 User-Agent: Notmuch/0.20.2+60~gcb08a2e (http://notmuchmail.org) Emacs/24.5.1\r
  32  (x86_64-pc-linux-gnu)\r
  33 Date: Sun, 30 Aug 2015 09:28:32 -0300\r
  34 Message-ID: <87y4gtrmdr.fsf@maritornes.cs.unb.ca>\r
  35 MIME-Version: 1.0\r
  36 Content-Type: text/plain\r
  37 Cc: tomi.ollila@iki.fi\r
  38 X-BeenThere: notmuch@notmuchmail.org\r
  39 X-Mailman-Version: 2.1.18\r
  40 Precedence: list\r
  41 List-Id: "Use and development of the notmuch mail system."\r
  42  <notmuch.notmuchmail.org>\r
  43 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  44  <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  45 List-Archive: <http://notmuchmail.org/pipermail/notmuch/>\r
  46 List-Post: <mailto:notmuch@notmuchmail.org>\r
  47 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  48 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  49  <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  50 X-List-Received-Date: Sun, 30 Aug 2015 12:29:52 -0000\r
  51 \r
  52 Tomi Ollila <tomi.ollila@iki.fi> writes:\r
  53 \r
  54 > +(defsubst notmuch-address--message-insinuated ()\r
  55 > +  (memq notmuch-address-message-alist-member message-completion-alist))\r
  56 > +\r
  57 \r
  58 Is there some advantage to defsubst other than (maybe?) performance?  It\r
  59 just seems like one more construct for people to get up to speed with\r
  60 the codebase.\r
  61 \r
  62 d\r