66/74f98af7eb8166b5613f79bbcd55b6dba9dc9f

   1 Return-Path: <cworth@cworth.org>\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 6F104476B9A\r
   6         for <notmuch@notmuchmail.org>; Tue,  9 Mar 2010 12:02:50 -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.022\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.022 tagged_above=-999 required=5\r
  12         tests=[ALL_TRUSTED=-1.8, AWL=-0.223, BAYES_50=0.001] 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 c+b+G5hU0Ks0; Tue,  9 Mar 2010 12:02:49 -0800 (PST)\r
  16 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  17         by olra.theworths.org (Postfix) with ESMTP id C0B85476B97;\r
  18         Tue,  9 Mar 2010 12:02:49 -0800 (PST)\r
  19 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  20         id DE1F954C004; Tue,  9 Mar 2010 09:36:16 -0800 (PST)\r
  21 From: Carl Worth <cworth@cworth.org>\r
  22 To: Ruben Pollan <meskio@sindominio.net>, notmuch@notmuchmail.org\r
  23 In-Reply-To: <87pr67ofz2.fsf@yoom.home.cworth.org>\r
  24 References: <20091126202347.GA16654@blackspot>\r
  25         <1260364206-344-1-git-send-email-meskio@sindominio.net>\r
  26         <20091209132446.GC23878@blackspot>\r
  27         <877hsv6hbo.fsf@yoom.home.cworth.org>\r
  28         <87r5qnol78.fsf@yoom.home.cworth.org>\r
  29         <87pr67ofz2.fsf@yoom.home.cworth.org>\r
  30 Date: Tue, 09 Mar 2010 09:36:10 -0800\r
  31 Message-ID: <873a09jt2t.fsf@yoom.home.cworth.org>\r
  32 MIME-Version: 1.0\r
  33 Content-Type: multipart/signed; boundary="=-=-=";\r
  34         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  35 Subject: Re: [notmuch] [PATCH] Added regress option to tags iterator\r
  36 X-BeenThere: notmuch@notmuchmail.org\r
  37 X-Mailman-Version: 2.1.13\r
  38 Precedence: list\r
  39 List-Id: "Use and development of the notmuch mail system."\r
  40         <notmuch.notmuchmail.org>\r
  41 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  42         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  43 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  44 List-Post: <mailto:notmuch@notmuchmail.org>\r
  45 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  46 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  47         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  48 X-List-Received-Date: Tue, 09 Mar 2010 20:02:50 -0000\r
  49 \r
  50 --=-=-=\r
  51 Content-Transfer-Encoding: quoted-printable\r
  52 \r
  53 On Mon, 21 Dec 2009 19:16:49 -0800, Carl Worth <cworth@cworth.org> wrote:\r
  54 > One option is to just drop the "move_ " prefix. Then everything will be\r
  55 > a two-word function. So the new proposal is:\r
  56 >=20\r
  57 > to_first\r
  58 > has_current\r
  59 > to_next\r
  60 >=20\r
  61 > to_last\r
  62 > has_current\r
  63 > to_previous\r
  64 >=20\r
  65 > get_current\r
  66 >=20\r
  67 > Better?\r
  68 \r
  69 Looking back at this proposal now, (after a long delay), I found I\r
  70 didn't like it at all. With function names like:\r
  71 \r
  72         notmuch_threads_to_first\r
  73         notmuch_threads_to_next\r
  74 \r
  75 the missing verb in the name is really distracting. I ended up reading\r
  76 these names as if they were conversion functions. So I've gone back to\r
  77 preferring the names with the explicit verbs (even though quite long):\r
  78 \r
  79         notmuch_threads_move_to_first\r
  80         notmuch_threads_move_to_next\r
  81 \r
  82 Meanwhile, I also decided that _current wasn't adding anything to the\r
  83 names it showed up in, (we can never "get" anything other than the\r
  84 current item, so why qualify as "get_current"?). So I'm changing from\r
  85 "has_current" to "valid", and leaving "get" as it is.\r
  86 \r
  87 So the final proposal for iteration in either direction is:\r
  88 \r
  89         move_to_first\r
  90         valid\r
  91         move_to_next\r
  92 \r
  93         move_to_last\r
  94         valid\r
  95         move_to_previous\r
  96 \r
  97         get\r
  98 \r
  99 I've just pushed commits changing the existing functions (which allow\r
 100 only forward iteration) to use this naming scheme. I haven't added any\r
 101 of the reverse-iteration functions yet, so Ruben, if you'd like to do\r
 102 those within this scheme, that would be find. (Or we could wait until we\r
 103 have an actual use in mind for them.)\r
 104 \r
 105 Thanks,\r
 106 \r
 107 =2DCarl\r
 108 \r
 109 --=-=-=\r
 110 Content-Type: application/pgp-signature\r
 111 \r
 112 -----BEGIN PGP SIGNATURE-----\r
 113 Version: GnuPG v1.4.10 (GNU/Linux)\r
 114 \r
 115 iD8DBQFLlocL6JDdNq8qSWgRAngCAKCOC1FhXomVg/nqQhct0ChFEXj/HgCfRAoj\r
 116 wJS2YBSFaiEi3Teya+Du/UI=\r
 117 =l7qE\r
 118 -----END PGP SIGNATURE-----\r
 119 --=-=-=--\r