1e/9918ab581ee12ff01276728ce8784738b72641

   1 Return-Path: <tomi.ollila@iki.fi>\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 6A0216DE1585\r
   6  for <notmuch@notmuchmail.org>; Sun, 25 Oct 2015 13:26:16 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at cworth.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: 1.118\r
  10 X-Spam-Level: *\r
  11 X-Spam-Status: No, score=1.118 tagged_above=-999 required=5 tests=[AWL=-0.278,\r
  12   SPF_NEUTRAL=0.652, URIBL_SBL=0.644, URIBL_SBL_A=0.1] 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 D1b1YAwxdL-N for <notmuch@notmuchmail.org>;\r
  16  Sun, 25 Oct 2015 13:26:14 -0700 (PDT)\r
  17 Received: from guru.guru-group.fi (guru.guru-group.fi [46.183.73.34])\r
  18  by arlo.cworth.org (Postfix) with ESMTP id BA21D6DE14FD\r
  19  for <notmuch@notmuchmail.org>; Sun, 25 Oct 2015 13:26:13 -0700 (PDT)\r
  20 Received: from guru.guru-group.fi (localhost [IPv6:::1])\r
  21  by guru.guru-group.fi (Postfix) with ESMTP id 79AF0100046;\r
  22  Sun, 25 Oct 2015 22:26:08 +0200 (EET)\r
  23 From: Tomi Ollila <tomi.ollila@iki.fi>\r
  24 To: David Bremner <david@tethera.net>, notmuch@notmuchmail.org\r
  25 Subject: Re: elisp + company completion patches, v7\r
  26 In-Reply-To: <1445786638-13763-1-git-send-email-david@tethera.net>\r
  27 References: <1445786638-13763-1-git-send-email-david@tethera.net>\r
  28 User-Agent: Notmuch/0.20.2+68~g0c35549 (http://notmuchmail.org) Emacs/24.3.1\r
  29  (x86_64-unknown-linux-gnu)\r
  30 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
  31  $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
  32  !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
  33 Date: Sun, 25 Oct 2015 22:26:08 +0200\r
  34 Message-ID: <m2lhaqu23j.fsf@guru.guru-group.fi>\r
  35 MIME-Version: 1.0\r
  36 Content-Type: text/plain\r
  37 X-BeenThere: notmuch@notmuchmail.org\r
  38 X-Mailman-Version: 2.1.20\r
  39 Precedence: list\r
  40 List-Id: "Use and development of the notmuch mail system."\r
  41  <notmuch.notmuchmail.org>\r
  42 List-Unsubscribe: <https://notmuchmail.org/mailman/options/notmuch>,\r
  43  <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  44 List-Archive: <http://notmuchmail.org/pipermail/notmuch/>\r
  45 List-Post: <mailto:notmuch@notmuchmail.org>\r
  46 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  47 List-Subscribe: <https://notmuchmail.org/mailman/listinfo/notmuch>,\r
  48  <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  49 X-List-Received-Date: Sun, 25 Oct 2015 20:26:16 -0000\r
  50 \r
  51 On Sun, Oct 25 2015, David Bremner <david@tethera.net> wrote:\r
  52 \r
  53 > This round contains several bug fixes from Mark, and a slightly\r
  54 > re-organized initialization from me.  In particular, TAB should now\r
  55 > behave sanely under company. Also there's a stub\r
  56 > notmuch-address-message-insinuate to prevent peoples emacs startup\r
  57 > from failing.\r
  58 \r
  59 Looks pretty good, I, like Mark, noticed some newline changes in patch 1\r
  60 (which is fixed in patch 2 so I personally would not care of that). The\r
  61 another indentation thing Mark noticed is easy to amend.\r
  62 \r
  63 I, if this does not sound nitpicking, would amend those 'This patch' and\r
  64 'With this patch' texts in commit messages away by some rewording -- those\r
  65 pose as a bad example for someone reading the code -- and diminish the\r
  66 effect when I complain that again in future patched ;)\r
  67 \r
  68 The basic completion with this new system using notmuch address output\r
  69 and (supposedly) caching the values seems to work very well and I believe\r
  70 is welcome feature to (new) users. I did not test how company mode works\r
  71 as I don't have it (and there is no company mode for this mixed environment\r
  72 what I use now -- except that I could install it from elpa; maybe I try\r
  73 that in near future...)\r
  74 \r
  75 BTW: if anyone knows how to enable lexical-binding to a block of code\r
  76 (i.e. alternative to -*- lexical-binding: t -*- (or can give hints, like\r
  77 s/\([^-]\)let/\1lexical-let/) I'd like to know...)\r
  78 \r
  79 Tomi\r