57/d8e0a246117b65238d03355f2deaf941e794ad

   1 Return-Path: <dmitry.kurochkin@gmail.com>\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 38AFC431FD0\r
   6         for <notmuch@notmuchmail.org>; Mon, 26 Dec 2011 03:10:45 -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.799\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.799 tagged_above=-999 required=5\r
  12         tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1,\r
  13         FREEMAIL_FROM=0.001, 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 DMchcx1QP98m for <notmuch@notmuchmail.org>;\r
  17         Mon, 26 Dec 2011 03:10:44 -0800 (PST)\r
  18 Received: from mail-lpp01m010-f53.google.com (mail-lpp01m010-f53.google.com\r
  19         [209.85.215.53]) (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 6FD86431FB6\r
  22         for <notmuch@notmuchmail.org>; Mon, 26 Dec 2011 03:10:44 -0800 (PST)\r
  23 Received: by lagr15 with SMTP id r15so5355697lag.26\r
  24         for <notmuch@notmuchmail.org>; Mon, 26 Dec 2011 03:10:42 -0800 (PST)\r
  25 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma;\r
  26         h=from:to:subject:in-reply-to:references:user-agent:date:message-id\r
  27         :mime-version:content-type;\r
  28         bh=dKc63NQ7HR1IC5Wnq4+0Q+EQpXYIpPanVtoqad0eHY4=;\r
  29         b=G7gulMprDPNlWnngZqLlojslWc74lpb7uGj7P0yQ80lo3aTvTVxdloCpbGR0T0nX07\r
  30         rkGS5VA11D0krJOaeQvvaLl7iNwAIPVTYnMqGei7JLEljvgt4K+1r/KBKocRQWeg1Gol\r
  31         xutFp+02xVizvZRwvuIqOGkFcCrjX1pdCBUPU=\r
  32 Received: by 10.152.133.70 with SMTP id pa6mr19474335lab.0.1324897842822;\r
  33         Mon, 26 Dec 2011 03:10:42 -0800 (PST)\r
  34 Received: from localhost ([194.186.188.211])\r
  35         by mx.google.com with ESMTPS id mg13sm980411lab.5.2011.12.26.03.10.41\r
  36         (version=TLSv1/SSLv3 cipher=OTHER);\r
  37         Mon, 26 Dec 2011 03:10:42 -0800 (PST)\r
  38 From: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>\r
  39 To: David Edmondson <dme@dme.org>, notmuch@notmuchmail.org\r
  40 Subject: Re: [RFC][PATCH v4] emacs: Re-implement advance/rewind functions of\r
  41         notmuch-show-mode.\r
  42 In-Reply-To: <cunpqfbtxu2.fsf@hotblack-desiato.hh.sledj.net>\r
  43 References: <id:"1324553312-10972-1-git-send-email-dme@dme.org">\r
  44         <1324665712-2419-1-git-send-email-dme@dme.org>\r
  45         <87ipl7kt82.fsf@gmail.com>\r
  46         <cunpqfbtxu2.fsf@hotblack-desiato.hh.sledj.net>\r
  47 User-Agent: Notmuch/0.10.2+135~gb811a3c (http://notmuchmail.org) Emacs/23.3.1\r
  48         (x86_64-pc-linux-gnu)\r
  49 Date: Mon, 26 Dec 2011 15:09:55 +0400\r
  50 Message-ID: <87fwg71tdo.fsf@gmail.com>\r
  51 MIME-Version: 1.0\r
  52 Content-Type: text/plain; charset=us-ascii\r
  53 X-BeenThere: notmuch@notmuchmail.org\r
  54 X-Mailman-Version: 2.1.13\r
  55 Precedence: list\r
  56 List-Id: "Use and development of the notmuch mail system."\r
  57         <notmuch.notmuchmail.org>\r
  58 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  59         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  60 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  61 List-Post: <mailto:notmuch@notmuchmail.org>\r
  62 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  63 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  64         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  65 X-List-Received-Date: Mon, 26 Dec 2011 11:10:45 -0000\r
  66 \r
  67 Hi David.\r
  68 \r
  69 On Mon, 26 Dec 2011 10:46:13 +0000, David Edmondson <dme@dme.org> wrote:\r
  70 > On Fri, 23 Dec 2011 23:01:33 +0400, Dmitry Kurochkin <dmitry.kurochkin@gmail.com> wrote:\r
  71 > > * Revert changes to notmuch-show-advance-and-archive.\r
  72 > \r
  73 > Why? (I mean, because the change is poor or just that it's unrelated or\r
  74 > because I didn't mention it)\r
  75 > \r
  76 \r
  77 Because it is unrelated.\r
  78 \r
  79 And can you please explain why `when' is better than `if' here?  Then I\r
  80 will know which one to use the next time :)\r
  81 \r
  82 > > * Can we split this in two patches?  One for rewind and another for\r
  83 > >   advance.\r
  84 > \r
  85 > I'll think about that. Is there a specific reason? I'm not particularly\r
  86 > in favour of splitting things just for the sake of it.\r
  87 > \r
  88 \r
  89 Because they are independent and can be split.  And it is easier to\r
  90 review (and work in general, I suppose) with two smaller patches than\r
  91 with a single bigger one.\r
  92 \r
  93 Though, since you got two other reviews already, you can just ignore\r
  94 this.\r
  95 \r
  96 > > * Does this patch change the behavior of the functions or is it just\r
  97 > >   meant to simplify the code?  If it is the former, it would be really\r
  98 > >   nice to have tests for it.\r
  99 > \r
 100 > I believe that it changes the behaviour. I'll write tests.\r
 101 \r
 102 Thanks.\r
 103 \r
 104 Regards,\r
 105   Dmitry\r