b8/37f157fbbccc51105fcf251ab4c4f6a00eed23

   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 376F2431FB6\r
   6         for <notmuch@notmuchmail.org>; Thu, 26 Jan 2012 08:19:01 -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 ptKEqi1X7WiE for <notmuch@notmuchmail.org>;\r
  17         Thu, 26 Jan 2012 08:19:00 -0800 (PST)\r
  18 Received: from mail-bk0-f53.google.com (mail-bk0-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 57CEB431FAE\r
  22         for <notmuch@notmuchmail.org>; Thu, 26 Jan 2012 08:19:00 -0800 (PST)\r
  23 Received: by bkbzt19 with SMTP id zt19so692127bkb.26\r
  24         for <notmuch@notmuchmail.org>; Thu, 26 Jan 2012 08:18:57 -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=4z8jf2u62haylsHcH8JKVLsfENc3bpPEPBtUZXSnIRQ=;\r
  29         b=OYqrgeIvJsnwlY52uZNnGR2o/q7R4Z+UBefVhxMhqt/E1GyBaVsFlmIxfuz7TlHNd9\r
  30         0jE1WcO1WtGWggZwbZUr3m7ATHGaLuKOtkiHBOidW6bBqBD6tfrkkCXnN1jASm1cwnjQ\r
  31         YrJVM3VzB79Sc3PuIy9FBDFGC3F4+1hdB8O6Y=\r
  32 Received: by 10.205.132.1 with SMTP id hs1mr939505bkc.45.1327594737597;\r
  33         Thu, 26 Jan 2012 08:18:57 -0800 (PST)\r
  34 Received: from localhost ([91.144.186.21])\r
  35         by mx.google.com with ESMTPS id ew13sm9999244bkb.1.2012.01.26.08.18.56\r
  36         (version=TLSv1/SSLv3 cipher=OTHER);\r
  37         Thu, 26 Jan 2012 08:18:56 -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: [PATCH 0/2] re-enable line wrapping and add some header bling\r
  41 In-Reply-To: <1327565871-19729-1-git-send-email-dme@dme.org>\r
  42 References: <1327565871-19729-1-git-send-email-dme@dme.org>\r
  43 User-Agent: Notmuch/0.11+116~ge6e10b8 (http://notmuchmail.org) Emacs/23.3.1\r
  44         (x86_64-pc-linux-gnu)\r
  45 Date: Thu, 26 Jan 2012 20:17:49 +0400\r
  46 Message-ID: <87ty3iqvyq.fsf@gmail.com>\r
  47 MIME-Version: 1.0\r
  48 Content-Type: text/plain; charset=us-ascii\r
  49 X-BeenThere: notmuch@notmuchmail.org\r
  50 X-Mailman-Version: 2.1.13\r
  51 Precedence: list\r
  52 List-Id: "Use and development of the notmuch mail system."\r
  53         <notmuch.notmuchmail.org>\r
  54 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  55         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  56 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  57 List-Post: <mailto:notmuch@notmuchmail.org>\r
  58 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  59 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  60         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  61 X-List-Received-Date: Thu, 26 Jan 2012 16:19:01 -0000\r
  62 \r
  63 Hi David.\r
  64 \r
  65 On Thu, 26 Jan 2012 08:17:49 +0000, David Edmondson <dme@dme.org> wrote:\r
  66 > By default, re-enable `visual-line-mode' in `notmuch-show-mode'. Do it\r
  67 > via a hook so that purists (ahem) can turn it off.\r
  68 > \r
  69 > Add some more processing of headers to make them look nice. Do it via\r
  70 > hooks so that unbelievers can turn it off.\r
  71 > \r
  72 \r
  73 I did not review the code, but here is a general comment for both\r
  74 patches (but especially for the first one).  It would be nice to have a\r
  75 more detailed documentation for hooks.  Docstring like "Enable Visual\r
  76 Line mode." for a function named `notmuch-show-turn-on-visual-line-mode'\r
  77 is near useless.  It is quite obvious that the function enables\r
  78 visual-line-mode from it's name.  And it does not give any information\r
  79 on why would someone actually want to use it.  I do not remember what\r
  80 visual-line-mode is exactly, so to understand whether this hook is\r
  81 actually useful for me, I have to read visual-line-mode docs, think\r
  82 about how it helps in notmuch-show, read some code, perhaps, etc.  I\r
  83 would argue that since the hook itself is trivial, the main point in\r
  84 having it is to provide a clearly documented solution for a common\r
  85 problem for those who do not know how to solve this problem right away.\r
  86 Currently, those who know what visual-line-mode is do not need this\r
  87 hook, because they can easily write their own, and those who do not know\r
  88 what visual-line-mode is can not use this hook, because it says nothing\r
  89 about why it is actually useful.\r
  90 \r
  91 Also, in addition to better docs, I would rename\r
  92 `notmuch-show-turn-on-visual-line-mode' to something that reflects what\r
  93 it does from user POV (like the other two hooks).\r
  94 \r
  95 Though, the fact that the hook is enabled by default makes the above\r
  96 arguments less important, I guess.\r
  97 \r
  98 Regards,\r
  99   Dmitry\r
 100 \r
 101 > David Edmondson (2):\r
 102 >   emacs: Re-enable line wrapping in `notmuch-show-mode'.\r
 103 >   emacs: Add more processing of displayed headers.\r
 104 > \r
 105 >  emacs/notmuch-show.el |   50 +++++++++++++++++++++++++++++++++++++++++-------\r
 106 >  1 files changed, 42 insertions(+), 8 deletions(-)\r
 107 > \r
 108 > -- \r
 109 > 1.7.8.3\r
 110 > \r
 111 > _______________________________________________\r
 112 > notmuch mailing list\r
 113 > notmuch@notmuchmail.org\r
 114 > http://notmuchmail.org/mailman/listinfo/notmuch\r