0d/c931b12f56197e37f848a9dd16f60040daa9b4

   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 24253431FD0\r
   6         for <notmuch@notmuchmail.org>; Tue, 12 Jul 2011 19:23:03 -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 B2xsdu+tBwKV for <notmuch@notmuchmail.org>;\r
  16         Tue, 12 Jul 2011 19:23:01 -0700 (PDT)\r
  17 Received: from dmz-mailsec-scanner-1.mit.edu (DMZ-MAILSEC-SCANNER-1.MIT.EDU\r
  18         [18.9.25.12])\r
  19         by olra.theworths.org (Postfix) with ESMTP id D9E49431FB6\r
  20         for <notmuch@notmuchmail.org>; Tue, 12 Jul 2011 19:23:00 -0700 (PDT)\r
  21 X-AuditID: 1209190c-b7c65ae00000117c-f1-4e1d018ca079\r
  22 Received: from mailhub-auth-3.mit.edu ( [18.9.21.43])\r
  23         by dmz-mailsec-scanner-1.mit.edu (Symantec Messaging Gateway) with SMTP\r
  24         id 6C.99.04476.C810D1E4; Tue, 12 Jul 2011 22:23:08 -0400 (EDT)\r
  25 Received: from outgoing.mit.edu (OUTGOING-AUTH.MIT.EDU [18.7.22.103])\r
  26         by mailhub-auth-3.mit.edu (8.13.8/8.9.2) with ESMTP id p6D2MxUA029011; \r
  27         Tue, 12 Jul 2011 22:22:59 -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 p6D2Mwp9016080\r
  32         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
  33         Tue, 12 Jul 2011 22:22:59 -0400 (EDT)\r
  34 Received: from amthrax by awakening.csail.mit.edu with local (Exim 4.72)\r
  35         (envelope-from <amdragon@mit.edu>)\r
  36         id 1Qgp67-0004kA-Lx; Tue, 12 Jul 2011 22:22:47 -0400\r
  37 Date: Tue, 12 Jul 2011 22:22:47 -0400\r
  38 From: Austin Clements <amdragon@MIT.EDU>\r
  39 To: Istvan Marko <notmuch@kismala.com>\r
  40 Subject: Re: Slowness (search opens every email file?)\r
  41 Message-ID: <20110713022247.GG25558@mit.edu>\r
  42 References: <20110711190721.GA5386@compy.jasonwoof.org>\r
  43         <CAH-f9WtBHJ8bL6eWoSNrTs=jkrOYYCuWEVVPDUt_iCw0dKcZbw@mail.gmail.com>\r
  44         <20110712202459.GB15019@compy.jasonwoof.org>\r
  45         <CAH-f9WujE8s3DK1PyDGf3kdWLx2qvPx=dp7WZ-z_v8prqauvug@mail.gmail.com>\r
  46         <m3pqlfhrkk.fsf@zsu.kismala.com>\r
  47 MIME-Version: 1.0\r
  48 Content-Type: text/plain; charset=us-ascii\r
  49 Content-Disposition: inline\r
  50 In-Reply-To: <m3pqlfhrkk.fsf@zsu.kismala.com>\r
  51 User-Agent: Mutt/1.5.20 (2009-06-14)\r
  52 X-Brightmail-Tracker:\r
  53  H4sIAAAAAAAAA+NgFmpmleLIzCtJLcpLzFFi42IR4hTV1u1hlPUz2H/cwOLlbm2L6zdnMjsw\r
  54         eXxe9JTR49mqW8wBTFFcNimpOZllqUX6dglcGb1fVrMVtHBVHL72kb2B8Sd7FyMnh4SAicSC\r
  55         JdOhbDGJC/fWs3UxcnEICexjlHjS/IgFwtnAKLFz6U5GCOckk8T6Jw+ZIJwljBK3zzewgvSz\r
  56         CKhKnJvwiwXEZhPQkNi2fzlQBweHiICaxJNn+SBhZgEtia0bPzCC2MICZhJX970Cs3kFdCSO\r
  57         tJ2D2jadSeLryn4miISgxMmZT1hgmm/8e8kEMpNZQFpi+T8OkDCngK5Ew/9PbCC2qICKxLX9\r
  58         7WwTGIVmIemehaR7FkL3AkbmVYyyKblVurmJmTnFqcm6xcmJeXmpRbqGermZJXqpKaWbGMFh\r
  59         Lcmzg/HNQaVDjAIcjEo8vLLxMn5CrIllxZW5hxglOZiURHkdGGT9hPiS8lMqMxKLM+KLSnNS\r
  60         iw8xSnAwK4nwrv8NVM6bklhZlVqUD5OS5mBREuct9/7vKySQnliSmp2aWpBaBJOV4eBQkuA9\r
  61         DDJUsCg1PbUiLTOnBCHNxMEJMpwHaPg6kBre4oLE3OLMdIj8KUZFKXHeySAJAZBERmkeXC8s\r
  62         7bxiFAd6RZh3M0gVDzBlwXW/AhrMBDT4tbQkyOCSRISUVANjbcvm2XWHvfY+43Lc87Va9Od2\r
  63         oVqZMNZLS9quOd+2PXTWsojvwmGuhDuSvyfKa5uwKEpt8TO1L3I/7l2yTbV4c/oF5YpJkV+f\r
  64         afqw5XFev6jsqepfa/nxy/e8eeeXzre4fuPBzMey63Me/ZLh2nxJxnGHzHdjWXND99W9nEc6\r
  65         M4qOarvvbjNUYinOSDTUYi4qTgQAEk+d4xYDAAA=\r
  66 Cc: Notmuch Mail <notmuch@notmuchmail.org>\r
  67 X-BeenThere: notmuch@notmuchmail.org\r
  68 X-Mailman-Version: 2.1.13\r
  69 Precedence: list\r
  70 List-Id: "Use and development of the notmuch mail system."\r
  71         <notmuch.notmuchmail.org>\r
  72 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  73         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  74 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  75 List-Post: <mailto:notmuch@notmuchmail.org>\r
  76 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  77 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  78         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  79 X-List-Received-Date: Wed, 13 Jul 2011 02:23:03 -0000\r
  80 \r
  81 Quoth Istvan Marko on Jul 12 at  6:55 pm:\r
  82 > Austin Clements <amdragon-3s7WtUTddSA@public.gmane.org> writes:\r
  83 > \r
  84 > > Istvan, did you make any progress on this patch since the last\r
  85 > > version?  I seem to recall it just needed general cleanup (code style\r
  86 > > and such) and a better answer for backwards compatibility (the\r
  87 > > unfortunate " " thing).\r
  88 > \r
  89 > I have been using the version that encodes empty headers to " " but\r
  90 > another way to handle this is to simply not set a VALUE for empty\r
  91 > headers and then fall back to the original parsing method for these\r
  92 > messages. Emails without from/subject/message-id headers are not very\r
  93 > common so perhaps this is a good compromise.\r
  94 > \r
  95 > Below is the patch without the " " hack.\r
  96 \r
  97 Ah, clever.  I was going to suggest adding another value to indicate\r
  98 the presence or absence of these Xapian values, but I like your\r
  99 solution better.  The only downside I can think of is that it might\r
 100 not extend to other headers if we store more header values in the\r
 101 database in the future.\r
 102 \r
 103 I'd say this patch looks good other than coding style\r
 104 - Tab indentation\r
 105 - /* */ comments, starting with a capital letter\r
 106 - Space between function name and open paren\r
 107 - Space after comma in argument lists\r
 108 - Spaces around assignment operator\r