3a/f9efff3a49b3422db57a2b9904551b5a976b78

   1 Return-Path: <amdragon@mit.edu>\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 15191431FAF\r
   6         for <notmuch@notmuchmail.org>; Wed, 22 Aug 2012 18:09:27 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -0.7\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_LOW=-0.7] 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 QBNzjSW5ysxt for <notmuch@notmuchmail.org>;\r
  16         Wed, 22 Aug 2012 18:09:26 -0700 (PDT)\r
  17 Received: from dmz-mailsec-scanner-4.mit.edu (DMZ-MAILSEC-SCANNER-4.MIT.EDU\r
  18         [18.9.25.15])\r
  19         by olra.theworths.org (Postfix) with ESMTP id 6C7CA431FAE\r
  20         for <notmuch@notmuchmail.org>; Wed, 22 Aug 2012 18:09:26 -0700 (PDT)\r
  21 X-AuditID: 1209190f-b7f306d0000008b4-33-503582c52b3e\r
  22 Received: from mailhub-auth-1.mit.edu ( [18.9.21.35])\r
  23         by dmz-mailsec-scanner-4.mit.edu (Symantec Messaging Gateway) with SMTP\r
  24         id 3F.71.02228.5C285305; Wed, 22 Aug 2012 21:09:25 -0400 (EDT)\r
  25 Received: from outgoing.mit.edu (OUTGOING-AUTH.MIT.EDU [18.7.22.103])\r
  26         by mailhub-auth-1.mit.edu (8.13.8/8.9.2) with ESMTP id q7N19Omm019691; \r
  27         Wed, 22 Aug 2012 21:09:25 -0400\r
  28 Received: from awakening.csail.mit.edu (awakening.csail.mit.edu [18.26.4.91])\r
  29         (authenticated bits=0)\r
  30         (User authenticated as amdragon@ATHENA.MIT.EDU)\r
  31         by outgoing.mit.edu (8.13.6/8.12.4) with ESMTP id q7N19N6u017455\r
  32         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
  33         Wed, 22 Aug 2012 21:09:24 -0400 (EDT)\r
  34 Received: from amthrax by awakening.csail.mit.edu with local (Exim 4.77)\r
  35         (envelope-from <amdragon@MIT.EDU>)\r
  36         id 1T4LvG-0000b0-Vc; Wed, 22 Aug 2012 21:09:23 -0400\r
  37 Date: Wed, 22 Aug 2012 21:09:22 -0400\r
  38 From: Austin Clements <amdragon@MIT.EDU>\r
  39 To: Bart Bunting <bart@ursys.com.au>\r
  40 Subject: Re: Errors after upgrade to 0.14\r
  41 Message-ID: <20120823010922.GV11179@mit.edu>\r
  42 References: <m2mx1m8pp3.fsf@ursys.com.au>\r
  43  <20120823004117.GT11179@mit.edu>\r
  44 MIME-Version: 1.0\r
  45 Content-Type: text/plain; charset=us-ascii\r
  46 Content-Disposition: inline\r
  47 In-Reply-To: <20120823004117.GT11179@mit.edu>\r
  48 User-Agent: Mutt/1.5.21 (2010-09-15)\r
  49 X-Brightmail-Tracker:\r
  50  H4sIAAAAAAAAA+NgFmpileLIzCtJLcpLzFFi42IR4hRV1j3aZBpgsGWxtcXFPfvZLa7fnMns\r
  51         wOTxbNUtZo9TXw6xBzBFcdmkpOZklqUW6dslcGU0n1/CVjBfpKLz0BHWBsYX/F2MnBwSAiYS\r
  52         +5puskHYYhIX7q0Hsrk4hAT2MUq8vPCCCcLZwChx+cNbqMxJJomnR++xQzhLGCXapk5lBOln\r
  53         EVCVePnvNAuIzSagIbFt/3KgOAeHiICKxJEdfCBhZgFpiW+/m5lAbGEBTYljPVdYQUp4BXQk\r
  54         zn3OBwkLCfhIrJy3hhnE5hUQlDg58wkLRKuWxI1/L5lAykHGLP/HARLmFNCVODnjDCuILQq0\r
  55         aMrJbWwTGIVmIemehaR7FkL3AkbmVYyyKblVurmJmTnFqcm6xcmJeXmpRbomermZJXqpKaWb\r
  56         GMFBLcm/g/HbQaVDjAIcjEo8vC/MTQOEWBPLiitzDzFKcjApifL+aAAK8SXlp1RmJBZnxBeV\r
  57         5qQWH2KU4GBWEuH9Uw2U401JrKxKLcqHSUlzsCiJ815NuekvJJCeWJKanZpakFoEk5Xh4FCS\r
  58         4HUHRq+QYFFqempFWmZOCUKaiYMTZDgP0HBrkBre4oLE3OLMdIj8KUZFKXHeCJCEAEgiozQP\r
  59         rheWdF4xigO9IgzRzgNMWHDdr4AGMwENVrtqDDK4JBEhJdXAmOiXNdXA4V77sV3Smy/cc3FQ\r
  60         Fa9U9m73sjn4dsHk4sT9qTd87BszNZxbFxf6Tmlm/OhmG2relKqq8yrkxsk7MXaba06cvOjw\r
  61         uWJ5KnMDq5Nk9R4e15DPxxz93hamtP5Zo37EaS3PBikPC/W53lsL8pq+Si6yWXPwg1Jb3ML/\r
  62         y3o+mtUyTJdVYinOSDTUYi4qTgQAcDmCWBUDAAA=\r
  63 Cc: notmuch@notmuchmail.org\r
  64 X-BeenThere: notmuch@notmuchmail.org\r
  65 X-Mailman-Version: 2.1.13\r
  66 Precedence: list\r
  67 List-Id: "Use and development of the notmuch mail system."\r
  68         <notmuch.notmuchmail.org>\r
  69 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  70         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  71 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  72 List-Post: <mailto:notmuch@notmuchmail.org>\r
  73 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  74 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  75         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  76 X-List-Received-Date: Thu, 23 Aug 2012 01:09:27 -0000\r
  77 \r
  78 Quoth myself on Aug 22 at  8:41 pm:\r
  79 > Quoth Bart Bunting on Aug 23 at  9:36 am:\r
  80 > > Good morning,\r
  81 > > \r
  82 > > After upgrading to notmuch 014 I am seeing the following messages appear\r
  83 > > in the messages buffer.\r
  84 > > \r
  85 > > error in process filter: byte-code: Wrong type argument: number-or-marker-p, nil\r
  86 > > error in process filter: Wrong type argument: number-or-marker-p, nil\r
  87 > > \r
  88 > > I am also getting a repeating message in the minibuffer (I think) which\r
  89 > > says something like "json read tail error".  Sorry that I am not more\r
  90 > > specific as I use emacspeak and this message appears to repeat many\r
  91 > > times interupting speech so I am not 100% sure of what it exactly says.\r
  92 > \r
  93 > This is probably "json-readtable-error", which is, unfortunately,\r
  94 > about the most generic error the JSON parser can give.\r
  95 > \r
  96 > > My gut feeling is that it is happening when notmuch is updating the\r
  97 > > database or something.\r
  98 > > \r
  99 > > Is this expected behaviour?  It is particularly annoying for me as it\r
 100 > > sends the speech synth crazy and crashes it for a period of about 30\r
 101 > > seconds.\r
 102 > > \r
 103 > > If it is expected then I will try and find a way to prevent emacspeak\r
 104 > > from trying to read it.\r
 105 > \r
 106 > This is definitely not expected behavior.  Does this happen when\r
 107 > you're searching for messages or when you're viewing a thread?  Can\r
 108 > you give any more details on what you're doing when you get this\r
 109 > error?\r
 110 > \r
 111 > Try doing M-x toggle-debug-on-error and then triggering the error.\r
 112 > Hopefully Emacs will give you a buffer with a backtrace that will give\r
 113 > us a better idea of where this is happening.\r
 114 \r
 115 Actually, I might know what's going on here.  Based on your suspicion\r
 116 about notmuch updating the database and assuming that this happens in\r
 117 the search buffer, I think the parser error recovery code is leaving\r
 118 the parser in a slightly invalid state, which causes the next\r
 119 invocation to think it can consume more data when there is no more\r
 120 data to consume.  I would expect that to give at most one readtable\r
 121 error, but maybe there's something I'm overlooking.\r
 122 \r
 123 Could you try the following one line patch?\r
 124 \r
 125 diff --git a/emacs/notmuch-lib.el b/emacs/notmuch-lib.el\r
 126 index 900235b..a09c0f6 100644\r
 127 --- a/emacs/notmuch-lib.el\r
 128 +++ b/emacs/notmuch-lib.el\r
 129 @@ -375,7 +375,7 @@ resynchronize after an error by moving point."\r
 130  \r
 131    (if (eq (notmuch-json-next jp) 'value)\r
 132        ;; We're already at a value\r
 133 -      nil\r
 134 +      (if (eobp) 'retry nil)\r
 135      ;; Drive the state toward 'expect-value\r
 136      (skip-chars-forward " \t\r\n")\r
 137      (or (when (eobp) 'retry)\r
 138 \r