f0/517fe3a773cc31f330434b58477da4399cd716

   1 Return-Path: <david@tethera.net>\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 12482431FC0\r
   6         for <notmuch@notmuchmail.org>; Mon, 14 Jul 2014 18:07:53 -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\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0 tagged_above=-999 required=5 tests=[none]\r
  12         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 Iq3EnedExGyv for <notmuch@notmuchmail.org>;\r
  16         Mon, 14 Jul 2014 18:07:49 -0700 (PDT)\r
  17 Received: from yantan.tethera.net (yantan.tethera.net [199.188.72.155])\r
  18         (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 9F0E8431FBC\r
  21         for <notmuch@notmuchmail.org>; Mon, 14 Jul 2014 18:07:49 -0700 (PDT)\r
  22 Received: from remotemail by yantan.tethera.net with local (Exim 4.80)\r
  23         (envelope-from <david@tethera.net>)\r
  24         id 1X6rDc-0006j4-EW; Mon, 14 Jul 2014 22:07:44 -0300\r
  25 Received: (nullmailer pid 8159 invoked by uid 1000); Tue, 15 Jul 2014\r
  26         01:07:40 -0000\r
  27 From: David Bremner <david@tethera.net>\r
  28 To: Mark Walters <markwalters1009@gmail.com>, notmuch@notmuchmail.org\r
  29 Subject: Re: [PATCH] emacs: tree/show remove duplicate function\r
  30 In-Reply-To: <1402134668-18587-1-git-send-email-markwalters1009@gmail.com>\r
  31 References: <1402134668-18587-1-git-send-email-markwalters1009@gmail.com>\r
  32 User-Agent: Notmuch/0.18.1+37~gde262a2 (http://notmuchmail.org) Emacs/24.3.1\r
  33         (x86_64-pc-linux-gnu)\r
  34 Date: Mon, 14 Jul 2014 22:07:40 -0300\r
  35 Message-ID: <87a98bbfab.fsf@maritornes.cs.unb.ca>\r
  36 MIME-Version: 1.0\r
  37 Content-Type: text/plain\r
  38 X-BeenThere: notmuch@notmuchmail.org\r
  39 X-Mailman-Version: 2.1.13\r
  40 Precedence: list\r
  41 List-Id: "Use and development of the notmuch mail system."\r
  42         <notmuch.notmuchmail.org>\r
  43 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  44         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  45 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  46 List-Post: <mailto:notmuch@notmuchmail.org>\r
  47 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  48 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  49         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  50 X-List-Received-Date: Tue, 15 Jul 2014 01:07:53 -0000\r
  51 \r
  52 Mark Walters <markwalters1009@gmail.com> writes:\r
  53 > +                (cond ((eq major-mode 'notmuch-show-mode)\r
  54 > +                       (notmuch-show-get-message-properties))\r
  55 > +                      ((eq major-mode 'notmuch-tree-mode)\r
  56 > +                       (notmuch-tree-get-message-properties))))\r
  57 \r
  58 I see this already existed, but it looks weird to me to have a two test\r
  59 cond with no else. Is it intentional to have the code drop through and\r
  60 do nothing if neither case matches?  It seems like it might be better to\r
  61 signal an error.\r
  62 \r
  63 d\r