06/6118dbdb15e1746605f15cf66eec711497197e

   1 Return-Path: <jrollins@finestructure.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 99192431FD0\r
   6         for <notmuch@notmuchmail.org>; Mon, 31 Oct 2011 14:44:37 -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.29\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.29 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_MED=-2.3, T_MIME_NO_TEXT=0.01] 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 vT0FPhfvPxpB for <notmuch@notmuchmail.org>;\r
  16         Mon, 31 Oct 2011 14:44:37 -0700 (PDT)\r
  17 Received: from outgoing-mail.its.caltech.edu (outgoing-mail.its.caltech.edu\r
  18         [131.215.239.19])\r
  19         by olra.theworths.org (Postfix) with ESMTP id 05FFD431FB6\r
  20         for <notmuch@notmuchmail.org>; Mon, 31 Oct 2011 14:44:37 -0700 (PDT)\r
  21 Received: from earth-doxen.imss.caltech.edu (localhost [127.0.0.1])\r
  22         by earth-doxen-postvirus (Postfix) with ESMTP id 884A766E02C3;\r
  23         Mon, 31 Oct 2011 14:44:36 -0700 (PDT)\r
  24 X-Spam-Scanned: at Caltech-IMSS on earth-doxen by amavisd-new\r
  25 Received: from finestructure.net (gwave-114.ligo.caltech.edu\r
  26  [131.215.114.114])     (Authenticated sender: jrollins)        by earth-doxen-submit\r
  27  (Postfix) with ESMTP id 63EB266E0198;  Mon, 31 Oct 2011 14:44:32 -0700 (PDT)\r
  28 Received: by finestructure.net (Postfix, from userid 1000)\r
  29         id 408BE4C3; Mon, 31 Oct 2011 14:44:32 -0700 (PDT)\r
  30 From: Jameson Graef Rollins <jrollins@finestructure.net>\r
  31 To: Jani Nikula <jani@nikula.org>, notmuch@notmuchmail.org\r
  32 Subject: Re: [RFC PATCH v2 0/3] lib/cli/emacs: limit number of messages in\r
  33         search results\r
  34 In-Reply-To: <cover.1320093940.git.jani@nikula.org>\r
  35 References: <cover.1320093940.git.jani@nikula.org>\r
  36 User-Agent: Notmuch/0.9+39~g74c9dba (http://notmuchmail.org) Emacs/23.3.1\r
  37         (x86_64-pc-linux-gnu)\r
  38 Date: Mon, 31 Oct 2011 14:44:29 -0700\r
  39 Message-ID: <87fwi86dky.fsf@servo.finestructure.net>\r
  40 MIME-Version: 1.0\r
  41 Content-Type: multipart/signed; boundary="=-=-=";\r
  42         micalg=pgp-sha256; protocol="application/pgp-signature"\r
  43 Cc: amdragon@mit.edu\r
  44 X-BeenThere: notmuch@notmuchmail.org\r
  45 X-Mailman-Version: 2.1.13\r
  46 Precedence: list\r
  47 List-Id: "Use and development of the notmuch mail system."\r
  48         <notmuch.notmuchmail.org>\r
  49 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  50         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  51 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  52 List-Post: <mailto:notmuch@notmuchmail.org>\r
  53 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  54 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  55         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  56 X-List-Received-Date: Mon, 31 Oct 2011 21:44:37 -0000\r
  57 \r
  58 --=-=-=\r
  59 Content-Transfer-Encoding: quoted-printable\r
  60 \r
  61 On Mon, 31 Oct 2011 23:18:07 +0200, Jani Nikula <jani@nikula.org> wrote:\r
  62 > Hi, this is an iteration of id:"cover.1319833617.git.jani@nikula.org" add=\r
  63 ressing\r
  64 > comments on the list and IRC. Main changes:\r
  65 >=20\r
  66 > * Results are now limited based on threads (not messages) for thread and =\r
  67 summary\r
  68 >   output. This is accomplished with a new lib function to count the numbe=\r
  69 r of\r
  70 >   threads in matching messages.\r
  71 >=20\r
  72 > * cli part is now inspired by James Vasile's patch\r
  73 >   id:"8739gyw0zh.fsf@opensourcematters.org", with the additional ability =\r
  74 to\r
  75 >   limit from the end of result set.\r
  76 >=20\r
  77 > * Bugs reported by Daniel Schoepe fixed.\r
  78 >=20\r
  79 > * Don't show buttons if everything is visible already.\r
  80 >=20\r
  81 > I'm still marking it as RFC. It works for me, but patch 1 might be deemed\r
  82 > unacceptable.\r
  83 \r
  84 Hi, Jani.  Thanks for working on this.  This should also be valuable for\r
  85 vim users.\r
  86 \r
  87 In order to push forward with this, though, I think we really need to\r
  88 have a complete unit test for this new functionality.  We usually like\r
  89 to see units tests that describe and then test for the new functionality\r
  90 you wish to add, followed by the patches that provide the new\r
  91 functionality.  Lots of good tests for new functionality being proposed\r
  92 here shouldn't be too difficult to work out ahead of time.\r
  93 \r
  94 For instance, here's an example of a test that I would like to see:\r
  95 \r
  96 test_begin_subtest "maxitems does the right thing"\r
  97 notmuch search tag:foo | head -n 20 >expected\r
  98 notmuch search --maxitems=3D20 tag:foo >output\r
  99 test_expect_equal_file expected output\r
 100 \r
 101 test_begin_subtest "concatenation of limited searches does the right thing"\r
 102 notmuch search tag:foo | head -n 20 >expected\r
 103 notmuch search --maxitems=3D10 tag:foo >output\r
 104 notmuch search --maxitems=3D10 --first=3D10 tag:foo >>output\r
 105 test_expect_equal_file expected output\r
 106 \r
 107 \r
 108 jamie.\r
 109 \r
 110 --=-=-=\r
 111 Content-Type: application/pgp-signature\r
 112 \r
 113 -----BEGIN PGP SIGNATURE-----\r
 114 Version: GnuPG v1.4.11 (GNU/Linux)\r
 115 \r
 116 iQIcBAEBCAAGBQJOrxa9AAoJEO00zqvie6q8Yr8QALccRpFWhrkKtMw7qPhjlmVE\r
 117 TBIWnNLXJYnO1YnaHhl90ebaizsV6b+3Cql6BD40V6yflisAUICr196FSXvsqo3q\r
 118 6fS0UoS65OLZNKL0EUmQp00BxKGsuJWyHfLVWZixMPnzGiLcrRJbzn8C0S2UWbVN\r
 119 cqMGyarjD4SJbLzqNAXyV9MACUavZ8npKoNa4dpaieB/54sYYxyzjvnnAnQi1012\r
 120 764DNWAJcwUB4EbNuMpmtOfyqve23dHhOVGRwB2qAZBYdFIkRvNig3ONx2tzvX9x\r
 121 x2DxoXKWu3NKnn5OX1u96HGIWT08/2ZqUvfkB8J6zOCrT1pt7b4nNfvrZPxEAEVl\r
 122 E1grle3Bd0p+NjxBEJTfJhQ3LGS6bzP4Y1Jtwmg5/VtxPQeuh1ouqq9pfRGcdV4e\r
 123 DMVnk8WG9YyCdcGiBXxz31zy6rHcO6+1JebRD/pnjyoDURd6jdIiBKHNou6uM2TM\r
 124 lxkoWC5i3wQk4OMxZvC2BKz38pAmambUD+M9rJeOpQv3FGnBaSPYJCT+cRA3X8JT\r
 125 laV6gthcoOig/Vo6t8pUYxy5XI5sRzPjTyGskUT+c0ea+VXSx6DhZLliQqYzDwSj\r
 126 FDVAmRz9WgFEdY89EwRymNTODfPE/Z/q8Mr6Il0U496rCBp4jkGAUceBp/KcNLX4\r
 127 dmYUYoQLHfgkQT9BsPFn\r
 128 =X+DZ\r
 129 -----END PGP SIGNATURE-----\r
 130 --=-=-=--\r