76/855f53cb6b9a1d3733e79bb5cf3f6cc3e9eda9

   1 Return-Path: <prvs=jrosenthal=7218428b9@jhu.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 011A94196F3\r
   6         for <notmuch@notmuchmail.org>; Thu, 22 Apr 2010 07:21:14 -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: -2.301\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.301 tagged_above=-999 required=5\r
  12         tests=[BAYES_20=-0.001, RCVD_IN_DNSWL_MED=-2.3] autolearn=ham\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 u6Ve1vJrl0Sr for <notmuch@notmuchmail.org>;\r
  16         Thu, 22 Apr 2010 07:21:13 -0700 (PDT)\r
  17 Received: from ipex3.johnshopkins.edu (ipex3.johnshopkins.edu\r
  18         [128.220.161.140])\r
  19         by olra.theworths.org (Postfix) with ESMTP id 2C5FC431FC1\r
  20         for <notmuch@notmuchmail.org>; Thu, 22 Apr 2010 07:21:13 -0700 (PDT)\r
  21 X-IronPort-AV: E=Sophos;i="4.52,257,1270440000"; d="scan'208";a="378222304"\r
  22 Received: from c-69-255-36-229.hsd1.md.comcast.net (HELO lucky)\r
  23         ([69.255.36.229])\r
  24         by ipex3.johnshopkins.edu with ESMTP/TLS/AES256-SHA;\r
  25         22 Apr 2010 10:21:12 -0400\r
  26 Received: from jkr by lucky with local (Exim 4.69)\r
  27         (envelope-from <jrosenthal@jhu.edu>)\r
  28         id 1O4xHD-00065P-Cd; Thu, 22 Apr 2010 10:21:11 -0400\r
  29 From: Jesse Rosenthal <jrosenthal@jhu.edu>\r
  30 To: cworth@cworth.org, Sebastian Spaeth <Sebastian@SSpaeth.de>,\r
  31         Notmuch development list <notmuch@notmuchmail.org>\r
  32 Subject: Re: sort order regression\r
  33 In-Reply-To: <87wrvz7ex3.fsf@jhu.edu>\r
  34 References: <87bpdbmvj0.fsf@SSpaeth.de> <87wrvz7ex3.fsf@jhu.edu>\r
  35 Date: Thu, 22 Apr 2010 10:21:11 -0400\r
  36 Message-ID: <87vdbj7elk.fsf@jhu.edu>\r
  37 MIME-Version: 1.0\r
  38 Content-Type: text/plain; charset=us-ascii\r
  39 X-BeenThere: notmuch@notmuchmail.org\r
  40 X-Mailman-Version: 2.1.13\r
  41 Precedence: list\r
  42 List-Id: "Use and development of the notmuch mail system."\r
  43         <notmuch.notmuchmail.org>\r
  44 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  45         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  46 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  47 List-Post: <mailto:notmuch@notmuchmail.org>\r
  48 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  49 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  50         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  51 X-List-Received-Date: Thu, 22 Apr 2010 14:21:14 -0000\r
  52 \r
  53 On Thu, 22 Apr 2010 10:14:16 -0400, Jesse Rosenthal <jrosenthal@jhu.edu> wrote:\r
  54 > Just to follow up on this, it seems that the regression comes from the\r
  55 > fix Carl introduced in 2a1a4f0551 to make his simplification of my patch\r
  56 > (simplification = 36e4459a3 , my patch = 4971b85641) pass tests. The\r
  57 > question is whether my original, more complicated version would have\r
  58 > passed the tests without the regressing fix.\r
  59 \r
  60 Okay, I just tested using the current test suite: my patch (4971b85641)\r
  61 passes all tests, while Carl's simplification fails.\r
  62 \r
  63 My suggestion would be to revert both the simplification and the fix to\r
  64 enable the simplification to pass: (36e4459a3, 2a1a4f0551).\r
  65 \r
  66 Should we send reverts as patches?\r
  67 \r
  68 Best,\r
  69 Jesse\r