4a/f99ca7fd70c8cf99954e80c7a8e6924a44d661

   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 olra.theworths.org (Postfix) with ESMTP id 02645431FC2\r
   6         for <notmuch@notmuchmail.org>; Sat, 17 Jan 2015 04:29:41 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: 2.438\r
  10 X-Spam-Level: **\r
  11 X-Spam-Status: No, score=2.438 tagged_above=-999 required=5\r
  12         tests=[DNS_FROM_AHBL_RHSBL=2.438] autolearn=disabled\r
  13 Received: from olra.theworths.org ([127.0.0.1])\r
  14         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15         with ESMTP id 8riIKuVUO6ph for <notmuch@notmuchmail.org>;\r
  16         Sat, 17 Jan 2015 04:29:37 -0800 (PST)\r
  17 Received: from yantan.tethera.net (yantan.tethera.net [199.188.72.155])\r
  18         (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id CC636431FAF\r
  21         for <notmuch@notmuchmail.org>; Sat, 17 Jan 2015 04:29:37 -0800 (PST)\r
  22 Received: from remotemail by yantan.tethera.net with local (Exim 4.80)\r
  23         (envelope-from <david@tethera.net>)\r
  24         id 1YCSVU-0006tU-RM; Sat, 17 Jan 2015 08:29:36 -0400\r
  25 Received: (nullmailer pid 25526 invoked by uid 1000); Sat, 17 Jan 2015\r
  26         12:29:31 -0000\r
  27 From: David Bremner <david@tethera.net>\r
  28 To: Gaute Hope <eg@gaute.vetsj.com>, notmuch <notmuch@notmuchmail.org>\r
  29 Subject: Re: DatabaseModifiedErrors causing troubles\r
  30 In-Reply-To: <1421493070-astroid-1-4x8pflg7mc-1327@strange>\r
  31 References:\r
  32  <CABKe4MvEdcsq8BZ-vq6R0Vnw87zEgBvqW_2F-Wysf5GNchqweg@mail.gmail.com>\r
  33         <87bnmkgr57.fsf@maritornes.cs.unb.ca>\r
  34         <1421493070-astroid-1-4x8pflg7mc-1327@strange>\r
  35 User-Agent: Notmuch/0.19+21~g71fb37d (http://notmuchmail.org) Emacs/24.4.1\r
  36         (x86_64-pc-linux-gnu)\r
  37 Date: Sat, 17 Jan 2015 13:29:31 +0100\r
  38 Message-ID: <87fvb9bnf8.fsf@maritornes.cs.unb.ca>\r
  39 MIME-Version: 1.0\r
  40 Content-Type: text/plain\r
  41 X-BeenThere: notmuch@notmuchmail.org\r
  42 X-Mailman-Version: 2.1.13\r
  43 Precedence: list\r
  44 List-Id: "Use and development of the notmuch mail system."\r
  45         <notmuch.notmuchmail.org>\r
  46 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  47         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  48 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  49 List-Post: <mailto:notmuch@notmuchmail.org>\r
  50 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  51 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  52         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  53 X-List-Received-Date: Sat, 17 Jan 2015 12:29:41 -0000\r
  54 \r
  55 Gaute Hope <eg@gaute.vetsj.com> writes:\r
  56 \r
  57 >\r
  58 > Hi David,\r
  59 >\r
  60 > Would it be possible with an error code that I could compare against in\r
  61 > stead? It would then work a bit like a global instance of the gmime\r
  62 > error. It could even be a preparation step against a gmime-error-style\r
  63 > solution in the far future.\r
  64 >\r
  65 > I am sure you know all the bad reasons for using a strcmp with strings\r
  66 > such as small (subtle) changes making them useless or future\r
  67 > localization of notmuch. This solution is in my opinion worse than the\r
  68 > current situation, it will lock things in and create problems for future\r
  69 > API compatability and application maintainers. I would rather wait for\r
  70 > or spend some time on a more general solution.\r
  71 \r
  72 I don't agree it's worse than the current situation but I take your\r
  73 point it isn't ideal.  We could do some kind "errno" in the database\r
  74 structure.  I think there are not that many functions with this\r
  75 unhelpful error return type. Based on a scan of notmuch.h, I see\r
  76 \r
  77 notmuch_query_search_threads\r
  78 notmuch_query_search_messages\r
  79 \r
  80 and the two count functions that I already posted API breaking patches\r
  81 for.  It might be better just to update the API (either adding versions\r
  82 with error returns, or just forcing people to change) for these\r
  83 functions.  Otherwise we have two different ways of returning status\r
  84 codes, and the "errno" is only used some of the time.\r
  85 \r
  86 d\r
  87 \r