2e/44772a77c8edbb9e0e120676dc21ee018ed943

   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 38BA0429E31\r
   6         for <notmuch@notmuchmail.org>; Sun,  5 Jun 2011 00:28:39 -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: -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 NJMs7uWRw6dg for <notmuch@notmuchmail.org>;\r
  17         Sun,  5 Jun 2011 00:28:38 -0700 (PDT)\r
  18 Received: from mail-bw0-f53.google.com (mail-bw0-f53.google.com\r
  19         [209.85.214.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 373B2431FB6\r
  22         for <notmuch@notmuchmail.org>; Sun,  5 Jun 2011 00:28:38 -0700 (PDT)\r
  23 Received: by bwg12 with SMTP id 12so2962834bwg.26\r
  24         for <notmuch@notmuchmail.org>; Sun, 05 Jun 2011 00:28:36 -0700 (PDT)\r
  25 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma;\r
  26         h=domainkey-signature:from:to:cc:subject:in-reply-to:references\r
  27         :user-agent:date:message-id:mime-version:content-type;\r
  28         bh=rE9M835XmOzN5/h81LAiIWJ2E7qzDwkHiuE3HLzBohk=;\r
  29         b=f3XNbueaTgD9t4akqAypatK7vokO0wIAu2PTKJRCEeNkjXY6JzLr/KcmcgzrrsXOwy\r
  30         OT3sjm2oi+pWnkNFadjWe0g5KplWReKSRGpFF3HE6Pmuq4i75KS9jGcLCyXT7oOsehcG\r
  31         WHrRL8Wu+7vBqXGcvdiXSai/KjAPmZxzoS0o4=\r
  32 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma;\r
  33         h=from:to:cc:subject:in-reply-to:references:user-agent:date\r
  34         :message-id:mime-version:content-type;\r
  35         b=YKYNRTyGwwEx0v0AwBfVslvgAis2Ctw3u2HtEcUpLnv/u1jMUefwjHrj9Ndj4Suomo\r
  36         wnhWFJfTdNhatSjajFU8fi2t+Ipb6FxSJ24mAElO0p8bBraZNt4sOiTYZcMfcT4dZnBE\r
  37         BDvkLXlr4I1FZEink4XFVtOrQbwIfEIPD+GHI=\r
  38 Received: by 10.205.83.68 with SMTP id af4mr3671481bkc.187.1307258916767;\r
  39         Sun, 05 Jun 2011 00:28:36 -0700 (PDT)\r
  40 Received: from localhost ([91.144.186.21])\r
  41         by mx.google.com with ESMTPS id ag6sm2437914bkc.6.2011.06.05.00.28.34\r
  42         (version=TLSv1/SSLv3 cipher=OTHER);\r
  43         Sun, 05 Jun 2011 00:28:35 -0700 (PDT)\r
  44 From: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>\r
  45 To: Dima Kogan <notmuch@dima.secretsauce.net>\r
  46 Subject: Re: [PATCH] Added C-up and C-down to cycle through previous searches\r
  47 In-Reply-To: <20110604130732.5d28fb68@smart>\r
  48 References: <1306659388-21809-1-git-send-email-notmuch@dima.secretsauce.net>\r
  49         <87vcwto5n3.fsf@gmail.com> <20110604130732.5d28fb68@smart>\r
  50 User-Agent: Notmuch/0.5-227-g031f722 (http://notmuchmail.org) Emacs/23.3.1\r
  51         (x86_64-pc-linux-gnu)\r
  52 Date: Sun, 05 Jun 2011 11:28:59 +0400\r
  53 Message-ID: <87fwnopwhw.fsf@gmail.com>\r
  54 MIME-Version: 1.0\r
  55 Content-Type: text/plain; charset=us-ascii\r
  56 Cc: notmuch@notmuchmail.org\r
  57 X-BeenThere: notmuch@notmuchmail.org\r
  58 X-Mailman-Version: 2.1.13\r
  59 Precedence: list\r
  60 List-Id: "Use and development of the notmuch mail system."\r
  61         <notmuch.notmuchmail.org>\r
  62 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  63         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  64 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  65 List-Post: <mailto:notmuch@notmuchmail.org>\r
  66 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  67 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  68         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  69 X-List-Received-Date: Sun, 05 Jun 2011 07:28:39 -0000\r
  70 \r
  71 Hi Dima.\r
  72 \r
  73 On Sat, 4 Jun 2011 13:07:32 -0700, Dima Kogan <notmuch@dima.secretsauce.net> wrote:\r
  74 > > On Sun, 29 May 2011 20:04:00 +0400\r
  75 > > Dmitry Kurochkin <dmitry.kurochkin@gmail.com> wrote:\r
  76 > >\r
  77 > > Hi Dima.\r
  78 > > \r
  79 > > On Sun, 29 May 2011 01:56:28 -0700, notmuch@dima.secretsauce.net\r
  80 > > wrote:\r
  81 > > > From: Dima Kogan <dima@secretsauce.net>\r
  82 > > > \r
  83 > > > ---\r
  84 > > > \r
  85 > > >  Hi.\r
  86 > > > \r
  87 > > >  I made a few improvements to the emacs UI. This patch allows the\r
  88 > > > user to scroll through the most recent searches with C-up and\r
  89 > > > C-down while in the search box.\r
  90 > > > \r
  91 > > \r
  92 > > This sounds like a very nice improvement to me!  I just have one\r
  93 > > concern: Why C-up and C-down?  I believe M-p and M-n would be more\r
  94 > > natural for Emacs users (at least for me :)).\r
  95 > > \r
  96 > > Also, I did not read the code and can not comment on it.  But notmuch\r
  97 > > coding style is to use tabs for indentation.  I guess .dir-locals.el\r
  98 > > should be improved to set coding style variables for all modes, not\r
  99 > > just C.\r
 100 > > \r
 101 > > Regards,\r
 102 > >   Dmitry\r
 103 > \r
 104 > Hi. I have a mild preference to C-up/down, but M-n/p is just fine also.\r
 105 > As for the coding style, I've looked through that file\r
 106 > (notmuch-hello.el), and it doesn't seem to have a consistent style as\r
 107 > far as tabs/spaces go. I'm happy to change my patch to conform, but I\r
 108 > don't know what I should be conforming to. Should I be tab-ing all\r
 109 > indents, or just some?\r
 110 > \r
 111 \r
 112 Notmuch uses a mix of 8 char width tabs and spaces.  First go tabs, then,\r
 113 if you need indenting with more precision, spaces.  Look at the lines\r
 114 your patch removes for an example.\r
 115 \r
 116 Also, .dir-locals.el sets some variables to configure Emacs for Notmuch\r
 117 coding style but only for c-mode.  You may set it by hand for lisp.\r
 118 \r
 119 Regards,\r
 120   Dmitry\r