a3/93a0b62472ea3db097f88f97a022319b1aa941

   1 Return-Path: <meskio@sindominio.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 74C0F431FAE\r
   6         for <notmuch@notmuchmail.org>; Sun,  6 Dec 2009 09:32:39 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 Received: from olra.theworths.org ([127.0.0.1])\r
   9         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  10         with ESMTP id 2uvjK5UINMo7 for <notmuch@notmuchmail.org>;\r
  11         Sun,  6 Dec 2009 09:32:37 -0800 (PST)\r
  12 Received: from flatline.sindominio.net (flatline.sindominio.net [82.144.4.26])\r
  13         by olra.theworths.org (Postfix) with ESMTP id 65376431FBC\r
  14         for <notmuch@notmuchmail.org>; Sun,  6 Dec 2009 09:32:37 -0800 (PST)\r
  15 Received: from localhost (localhost.localdomain [127.0.0.1])\r
  16         by flatline.sindominio.net (Postfix) with ESMTP id 9010D262AC6;\r
  17         Sun,  6 Dec 2009 18:32:35 +0100 (CET)\r
  18 X-Virus-Scanned: Debian amavisd-new at sindominio.net\r
  19 Received: from flatline.sindominio.net ([127.0.0.1])\r
  20         by localhost (flatline.sindominio.net [127.0.0.1]) (amavisd-new,\r
  21         port 10024)\r
  22         with ESMTP id xK4jWHdWtClY; Sun,  6 Dec 2009 18:32:31 +0100 (CET)\r
  23 Received: from blackspot (115.182.192-77.rev.gaoland.net [77.192.182.115])\r
  24         by flatline.sindominio.net (Postfix) with ESMTPA id 1651A262B5F;\r
  25         Sun,  6 Dec 2009 18:32:31 +0100 (CET)\r
  26 Received: by blackspot (Postfix, from userid 1000)\r
  27         id BF1318BC37; Sun,  6 Dec 2009 18:34:37 +0100 (CET)\r
  28 Date: Sun, 6 Dec 2009 18:34:37 +0100\r
  29 From: Ruben Pollan <meskio@sindominio.net>\r
  30 To: Carl Worth <cworth@cworth.org>\r
  31 Message-ID: <20091206173437.GD5109@blackspot>\r
  32 References: <20091126202347.GA16654@blackspot>\r
  33         <87pr73jo6w.fsf@yoom.home.cworth.org>\r
  34 MIME-Version: 1.0\r
  35 Content-Type: multipart/signed; micalg=pgp-sha1;\r
  36         protocol="application/pgp-signature"; boundary="iVCmgExH7+hIHJ1A"\r
  37 Content-Disposition: inline\r
  38 In-Reply-To: <87pr73jo6w.fsf@yoom.home.cworth.org>\r
  39 User-Agent: Mutt/1.5.20 (2009-06-14)\r
  40 Cc: notmuch@notmuchmail.org\r
  41 Subject: Re: [notmuch] notmuch_threads_back and notmuch_messages_back\r
  42 X-BeenThere: notmuch@notmuchmail.org\r
  43 X-Mailman-Version: 2.1.12\r
  44 Precedence: list\r
  45 List-Id: "Use and development of the notmuch mail system."\r
  46         <notmuch.notmuchmail.org>\r
  47 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  48         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  49 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  50 List-Post: <mailto:notmuch@notmuchmail.org>\r
  51 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  52 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  53         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  54 X-List-Received-Date: Sun, 06 Dec 2009 17:32:39 -0000\r
  55 \r
  56 \r
  57 --iVCmgExH7+hIHJ1A\r
  58 Content-Type: text/plain; charset=iso-8859-1\r
  59 Content-Disposition: inline\r
  60 Content-Transfer-Encoding: quoted-printable\r
  61 \r
  62 On 19:57, Fri 27 Nov 09, Carl Worth wrote:\r
  63 > (I'd like a verb that pairs better\r
  64 > with "advance" than the non-verb "back"---any suggestions)?\r
  65 \r
  66 What about regress?\r
  67 I'm not a native English speaker, so maybe someone can suggest something be=\r
  68 tter.\r
  69 \r
  70 > So those won't need any new code. The one case that will need new code\r
  71 > is that for notmuch_message_get_replies and\r
  72 > notmuch_message_get_toplevel_messages the messages iterator is currently\r
  73 > built on a singly-linked list. Making it doubly linked would obviously\r
  74 > not be hard though.\r
  75 \r
  76 I'm trying to convert it to a doubly linked list, but I'm not sure if I\r
  77 understand well how they work.\r
  78 \r
  79 I don't see how _notmuch_message_list_append works. It says "node can of co=\r
  80 urse\r
  81 point to an arbitrarily long list of nodes", but the code is:\r
  82 \r
  83 void\r
  84 _notmuch_message_list_append (notmuch_message_list_t *list,               =\r
  85 =20\r
  86                   notmuch_message_node_t *node)                           =\r
  87 =20\r
  88 {  =20\r
  89     *(list->tail) =3D node;\r
  90     list->tail =3D &node->next;\r
  91 }\r
  92 \r
  93 Should not be something like:\r
  94 \r
  95 void\r
  96 _notmuch_message_list_append (notmuch_message_list_t *list,               =\r
  97 =20\r
  98                   notmuch_message_node_t *node)                           =\r
  99 =20\r
 100 {  =20\r
 101     notmuch_message_node_t *next_node;\r
 102 \r
 103     *(list->tail) =3D node;\r
 104         for (next_node =3D node->next; next_node->next !=3D NULL;\r
 105              next_node =3D next_node->next);\r
 106         list->tail =3D &next_node->next;\r
 107 }\r
 108 \r
 109 Do I miss something? Or the function is just designed to work on a single n=\r
 110 ode\r
 111 not a list of them?\r
 112 \r
 113 \r
 114 --=20\r
 115 Rub=E9n Poll=E1n  | jabber:meskio@jabber.org\r
 116 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=\r
 117 =3D-=3D-=3D-=3D-=3D-\r
 118          No vamos a reivindicar nada,\r
 119             no vamos a pedir nada.\r
 120             Tomaremos, okuparemos.\r
 121 \r
 122 --iVCmgExH7+hIHJ1A\r
 123 Content-Type: application/pgp-signature; name="signature.asc"\r
 124 Content-Description: Digital signature\r
 125 Content-Disposition: inline\r
 126 \r
 127 -----BEGIN PGP SIGNATURE-----\r
 128 Version: GnuPG v1.4.10 (GNU/Linux)\r
 129 \r
 130 iEYEARECAAYFAksb6y0ACgkQGKOQ92SwNMKMgwCZAWxxhi/Q7JmGP9Ei8jNdn+tf\r
 131 OVoAmQE3utGJsdC8wH3KkkmpFPfZD/Ue\r
 132 =Rfin\r
 133 -----END PGP SIGNATURE-----\r
 134 \r
 135 --iVCmgExH7+hIHJ1A--\r