00/098607ba42257636ed6991b05a81a4b2ec055b

   1 Return-Path: <dme@dme.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 AB663431FBC\r
   6         for <notmuch@notmuchmail.org>; Mon, 30 Jan 2012 08:52:28 -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.7\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled\r
  13 Received: from olra.theworths.org ([127.0.0.1])\r
  14         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15         with ESMTP id I+IAEY+H-VlA for <notmuch@notmuchmail.org>;\r
  16         Mon, 30 Jan 2012 08:52:28 -0800 (PST)\r
  17 Received: from mail-ww0-f45.google.com (mail-ww0-f45.google.com\r
  18  [74.125.82.45])        (using TLSv1 with cipher RC4-SHA (128/128 bits))        (No client\r
  19  certificate requested) by olra.theworths.org (Postfix) with ESMTPS id\r
  20  07EB0431E64    for <notmuch@notmuchmail.org>; Mon, 30 Jan 2012 08:52:27 -0800\r
  21  (PST)\r
  22 Received: by wgbdt12 with SMTP id dt12so4222950wgb.2\r
  23         for <notmuch@notmuchmail.org>; Mon, 30 Jan 2012 08:52:26 -0800 (PST)\r
  24 Received: by 10.180.85.105 with SMTP id g9mr10128208wiz.12.1327942346821;\r
  25         Mon, 30 Jan 2012 08:52:26 -0800 (PST)\r
  26 Received: from hotblack-desiato.hh.sledj.net\r
  27         (host81-149-164-25.in-addr.btopenworld.com. [81.149.164.25])\r
  28         by mx.google.com with ESMTPS id eq5sm54211690wib.2.2012.01.30.08.52.25\r
  29         (version=TLSv1/SSLv3 cipher=OTHER);\r
  30         Mon, 30 Jan 2012 08:52:25 -0800 (PST)\r
  31 Received: by hotblack-desiato.hh.sledj.net (Postfix, from userid 30000)\r
  32         id B7C029FDD7; Mon, 30 Jan 2012 16:52:23 +0000 (GMT)\r
  33 From: David Edmondson <dme@dme.org>\r
  34 To: notmuch@notmuchmail.org\r
  35 Subject: [PATCH] emacs: Move the blank line from the bottom of the headers to\r
  36         the top of the body.\r
  37 Date: Mon, 30 Jan 2012 16:52:20 +0000\r
  38 Message-Id: <1327942340-23340-1-git-send-email-dme@dme.org>\r
  39 X-Mailer: git-send-email 1.7.8.3\r
  40 X-BeenThere: notmuch@notmuchmail.org\r
  41 X-Mailman-Version: 2.1.13\r
  42 Precedence: list\r
  43 List-Id: "Use and development of the notmuch mail system."\r
  44         <notmuch.notmuchmail.org>\r
  45 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  46         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  47 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  48 List-Post: <mailto:notmuch@notmuchmail.org>\r
  49 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  50 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  51         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  52 X-List-Received-Date: Mon, 30 Jan 2012 16:52:28 -0000\r
  53 \r
  54 The blank line doesn't really change position, but is now considered\r
  55 to be part of the body rather than part of the headers. This means\r
  56 that it is visible when the body is visible rather than when the\r
  57 headers are visible.\r
  58 ---\r
  59 \r
  60 I'm interested in getting feedback on this.\r
  61 \r
  62 Unless you run with `notmuch-message-headers-visible' set to `nil' or\r
  63 regularly toggle header visibility, you probably don't care.\r
  64 \r
  65  emacs/notmuch-show.el |    4 ++--\r
  66  1 files changed, 2 insertions(+), 2 deletions(-)\r
  67 \r
  68 diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el\r
  69 index 84ac624..73f73d4 100644\r
  70 --- a/emacs/notmuch-show.el\r
  71 +++ b/emacs/notmuch-show.el\r
  72 @@ -854,8 +854,6 @@ current buffer, if possible."\r
  73      ;; compatible with the existing implementation. This just sets it\r
  74      ;; to after the first header.\r
  75      (notmuch-show-insert-headers headers)\r
  76 -    ;; Headers should include a blank line (backwards compatibility).\r
  77 -    (insert "\n")\r
  78      (save-excursion\r
  79        (goto-char content-start)\r
  80        ;; If the subject of this message is the same as that of the\r
  81 @@ -870,6 +868,8 @@ current buffer, if possible."\r
  82      (setq notmuch-show-previous-subject bare-subject)\r
  83  \r
  84      (setq body-start (point-marker))\r
  85 +    ;; A blank line between the headers and the body.\r
  86 +    (insert "\n")\r
  87      (notmuch-show-insert-body msg (plist-get msg :body) depth)\r
  88      ;; Ensure that the body ends with a newline.\r
  89      (unless (bolp)\r
  90 -- \r
  91 1.7.8.3\r
  92 \r