9f/f4917bcfd327264c2385ec6db8a8901ab277ff

   1 Return-Path: <daniel@schoepe.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 F2672431FBC\r
   6         for <notmuch@notmuchmail.org>; Mon,  5 Mar 2012 03:17:55 -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: -0.8\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.8 tagged_above=-999 required=5\r
  12         tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1,\r
  13         RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled\r
  14 Received: from olra.theworths.org ([127.0.0.1])\r
  15         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  16         with ESMTP id sGCS-OhSGvAU for <notmuch@notmuchmail.org>;\r
  17         Mon,  5 Mar 2012 03:17:55 -0800 (PST)\r
  18 Received: from mail-ey0-f181.google.com (mail-ey0-f181.google.com\r
  19         [209.85.215.181]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
  20         (No client certificate requested)\r
  21         by olra.theworths.org (Postfix) with ESMTPS id DEAA2431FAE\r
  22         for <notmuch@notmuchmail.org>; Mon,  5 Mar 2012 03:17:54 -0800 (PST)\r
  23 Received: by eaa1 with SMTP id 1so1934172eaa.26\r
  24         for <notmuch@notmuchmail.org>; Mon, 05 Mar 2012 03:17:53 -0800 (PST)\r
  25 Received-SPF: pass (google.com: domain of daniel@schoepe.org designates\r
  26         10.14.200.132 as permitted sender) client-ip=10.14.200.132; \r
  27 Authentication-Results: mr.google.com;\r
  28         spf=pass (google.com: domain of daniel@schoepe.org\r
  29         designates 10.14.200.132 as permitted sender)\r
  30         smtp.mail=daniel@schoepe.org;\r
  31         dkim=pass header.i=daniel@schoepe.org\r
  32 Received: from mr.google.com ([10.14.200.132])\r
  33         by 10.14.200.132 with SMTP id z4mr11058657een.122.1330946273634\r
  34         (num_hops = 1); Mon, 05 Mar 2012 03:17:53 -0800 (PST)\r
  35 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=schoepe.org; s=google;\r
  36         h=from:to:subject:in-reply-to:references:user-agent:date:message-id\r
  37         :mime-version:content-type;\r
  38         bh=TNVaYUw/KRvzuOfAuR98uI98faZiW9jcMiGvlWWYgOs=;\r
  39         b=QelIH3apG5ANm8VgXwAdxlfM8/X7Grvx8dDpA5X8/HxyWBlC7MrWbkIOmbX75wJYAd\r
  40         0F4ha5cX9fS0JlXZ6coKXC5Q/7h8+ow8ITGTjDMpupgYQtFSVGBBIJmX3xjk17mclfLK\r
  41         I+hexbRxHAacz2m7OH/OkiRh4jQvJOSxCYetY=\r
  42 Received: by 10.14.200.132 with SMTP id z4mr8376564een.122.1330946273521;\r
  43         Mon, 05 Mar 2012 03:17:53 -0800 (PST)\r
  44 Received: from localhost (dslb-088-069-139-015.pools.arcor-ip.net.\r
  45         [88.69.139.15])\r
  46         by mx.google.com with ESMTPS id v51sm60569269eef.2.2012.03.05.03.17.51\r
  47         (version=TLSv1/SSLv3 cipher=OTHER);\r
  48         Mon, 05 Mar 2012 03:17:52 -0800 (PST)\r
  49 From: Daniel Schoepe <daniel@schoepe.org>\r
  50 To: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>, notmuch@notmuchmail.org\r
  51 Subject: Re: [PATCH v2] emacs: Pass a copy to\r
  52         notmuch-saved-search-sort-function\r
  53 In-Reply-To: <87booblrcv.fsf@gmail.com>\r
  54 References: <1330613059-5130-1-git-send-email-daniel@schoepe.org>\r
  55         <1330633478-1974-1-git-send-email-daniel@schoepe.org>\r
  56         <1330633478-1974-2-git-send-email-daniel@schoepe.org>\r
  57         <87booblrcv.fsf@gmail.com>\r
  58 User-Agent: Notmuch/0.11.1+212~gcd799cb (http://notmuchmail.org)\r
  59         Emacs/24.0.93.1 (x86_64-pc-linux-gnu)\r
  60 Date: Mon, 05 Mar 2012 12:17:43 +0100\r
  61 Message-ID: <87399n2t60.fsf@schoepe.localhost>\r
  62 MIME-Version: 1.0\r
  63 Content-Type: multipart/signed; boundary="=-=-=";\r
  64         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  65 X-Gm-Message-State:\r
  66  ALoCoQmidj6oU4mCSVbqoNVHCZj7hSlym6QzG/Z2rQ6OeujXqlAfPAqoPFbl/8NdFd+bXPFOEy8Z\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: Mon, 05 Mar 2012 11:17:56 -0000\r
  80 \r
  81 --=-=-=\r
  82 Content-Type: text/plain\r
  83 Content-Transfer-Encoding: quoted-printable\r
  84 \r
  85 On Mon, 05 Mar 2012 06:21:52 +0400, Dmitry Kurochkin <dmitry.kurochkin@gmai=\r
  86 l.com> wrote:\r
  87 > On Thu,  1 Mar 2012 21:24:38 +0100, Daniel Schoepe <daniel@schoepe.org> w=\r
  88 rote:\r
  89 > > notmuch-saved-search-sort-function might destructively modify its\r
  90 > > input (`sort' does that, for instance), so it should not be given\r
  91 > > notmuch-saved-searches directly.\r
  92 > > ---\r
  93 >=20\r
  94 > -1\r
  95 >=20\r
  96 > I think we should require `notmuch-saved-search-sort-function' not to\r
  97 > have side effects.  Current documentation should be more clear about\r
  98 > this.  We need to fix `notmuch-sort-saved-searches' to copy the list\r
  99 > before calling `sort'.  But we should not do it in\r
 100 > `notmuch-hello-insert-saved-searches' for any sorting function (which\r
 101 > may not need this copying).\r
 102 \r
 103 My reasoning was that since sort is such a common function, many users\r
 104 will probably use sort for their own sorting functions, not realizing\r
 105 that it has side effects. This will lead to confusing behavior that's\r
 106 not so easy to track down.\r
 107 \r
 108 Copying the list of saved searches when running notmuch-hello does not\r
 109 seem be relevant to performance to me, since it's a) not called that\r
 110 often and b) the list of saved searches will rarely exceed 30 elements.\r
 111 \r
 112 Hence, this way we can avoid some headaches for users who define their\r
 113 own sorting functions at a negligible (performance) cost. Incidentally,\r
 114 this is also how notmuch-hello did it before the user-defined sections\r
 115 patches.\r
 116 \r
 117 Cheers,\r
 118 Daniel\r
 119 \r
 120 --=-=-=\r
 121 Content-Type: application/pgp-signature\r
 122 \r
 123 -----BEGIN PGP SIGNATURE-----\r
 124 Version: GnuPG v1.4.12 (GNU/Linux)\r
 125 \r
 126 iQIcBAEBAgAGBQJPVKDXAAoJEIaTAtce+Z+JExkP/iM8DDMQ/QXJZzH/XTTvajXI\r
 127 cv1R0tG5gNfy88j+8Oy206ysSn9QO3ajUMZol3N7ZxBquHt1rvrwsKlkYgdi7DX0\r
 128 iPAw+fiutzEmCoac4kl2Tz0Boy8TXPEb47ZcnW8W6QdNxt0BOKX0fbg/dRgiYMVF\r
 129 s1TiyHyU+OnB7bIiUReum4F5VNN9bTxSD7vB/L+ciVEs1hGorBvPjBeDcr1hGsF6\r
 130 bmZXoKduNY8IX4hDXGTqEwieHmL7vzSHPdQzfkfjpcc+hOYAXOsTx6ht2pDS1JSa\r
 131 ko/by737i17Iyfz4ut7+q86H5YX00C3WmfIY6X5dIYgBADgQWkgGz6HRvYlkddIO\r
 132 G0zoqM5cQoif3MuRx/7i8MXGTnYTyXcYqbYDRz0OKyKIACMtiAJpVuwxqAEQu/Un\r
 133 FCqFmb1wGkCWYsY5LTCCPX1HrVnfmjFk5AzWmrRxE6DCSG5u9OSYlMmx+ddzohjN\r
 134 Nr3IfzPt9LmNxBZDamjk1103w+ZMckymHz6+XSux3gUi3xtRh17gj+eXrF4/Y/bD\r
 135 Sud2awOe0IRVpAF4+qsPTd+5fTUzSRVaV6ewOLRO1EBf2xeSRSBuLREVCmp51qak\r
 136 XB97KFOdhUd80PXZw7KY38EF4xJRZtdNXU8U3di6thhSGZX17uNVNbI6kPl/dzHX\r
 137 k5/3iYRNe3TudrKxej+W\r
 138 =VdWI\r
 139 -----END PGP SIGNATURE-----\r
 140 --=-=-=--\r