32/904e70ac23903d6eaf218022fd878ae3cd1e5f

   1 Return-Path: <bremner@tesseract.cs.unb.ca>\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 03C8B6DE01BA\r
   6  for <notmuch@notmuchmail.org>; Thu, 30 Jun 2016 01:40:46 -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.005\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.005 tagged_above=-999 required=5\r
  12  tests=[AWL=-0.006, HEADER_FROM_DIFFERENT_DOMAINS=0.001]\r
  13  autolearn=disabled\r
  14 Received: from arlo.cworth.org ([127.0.0.1])\r
  15  by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024)\r
  16  with ESMTP id xmWjzgRfTqWR for <notmuch@notmuchmail.org>;\r
  17  Thu, 30 Jun 2016 01:40:38 -0700 (PDT)\r
  18 Received: from fethera.tethera.net (fethera.tethera.net [198.245.60.197])\r
  19  by arlo.cworth.org (Postfix) with ESMTPS id 075D06DE00BF\r
  20  for <notmuch@notmuchmail.org>; Thu, 30 Jun 2016 01:40:37 -0700 (PDT)\r
  21 Received: from remotemail by fethera.tethera.net with local (Exim 4.84)\r
  22  (envelope-from <bremner@tesseract.cs.unb.ca>)\r
  23  id 1bIXWB-0007dx-Vn; Thu, 30 Jun 2016 04:40:16 -0400\r
  24 Received: (nullmailer pid 30879 invoked by uid 1000);\r
  25  Thu, 30 Jun 2016 08:40:30 -0000\r
  26 From: David Bremner <david@tethera.net>\r
  27 To: David Bremner <david@tethera.net>, notmuch@notmuchmail.org\r
  28 Subject: RFC: do what jamie wants with notmuch tag\r
  29 Date: Thu, 30 Jun 2016 10:40:25 +0200\r
  30 Message-Id: <1467276027-30633-1-git-send-email-david@tethera.net>\r
  31 X-Mailer: git-send-email 2.8.1\r
  32 In-Reply-To: <87fustdsx9.fsf@zancas.localnet>\r
  33 References: <87fustdsx9.fsf@zancas.localnet>\r
  34 X-BeenThere: notmuch@notmuchmail.org\r
  35 X-Mailman-Version: 2.1.20\r
  36 Precedence: list\r
  37 List-Id: "Use and development of the notmuch mail system."\r
  38  <notmuch.notmuchmail.org>\r
  39 List-Unsubscribe: <https://notmuchmail.org/mailman/options/notmuch>,\r
  40  <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  41 List-Archive: <http://notmuchmail.org/pipermail/notmuch/>\r
  42 List-Post: <mailto:notmuch@notmuchmail.org>\r
  43 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  44 List-Subscribe: <https://notmuchmail.org/mailman/listinfo/notmuch>,\r
  45  <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  46 X-List-Received-Date: Thu, 30 Jun 2016 08:40:46 -0000\r
  47 \r
  48 The second patch in particular could be cleaned up; there is definitely some\r
  49 duplication between the two code paths, so opening the database and\r
  50 creating a query could/should be refactored.\r
  51 \r
  52 It seems somewhat reasonable to have notmuch_query_search_tags in the\r
  53 API. I would say just document the limitation of ignoring excludes\r
  54 when querying for all tags, otherwise that optimization (which is a\r
  55 huge speed improvement) will mostly be useless.\r
  56 \r
  57 \r