c5/2c8864103b0729b39ca87df51888c547e10e64

   1 Return-Path: <bremner@unb.ca>\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 44541431FBF\r
   6         for <notmuch@notmuchmail.org>; Fri, 29 Mar 2013 18:48: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 W6WrAntZro7P for <notmuch@notmuchmail.org>;\r
  16         Fri, 29 Mar 2013 18:48:49 -0700 (PDT)\r
  17 Received: from tesseract.cs.unb.ca (tesseract.cs.unb.ca [131.202.240.238])\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 2E5BB431FAE\r
  21         for <notmuch@notmuchmail.org>; Fri, 29 Mar 2013 18:48:49 -0700 (PDT)\r
  22 Received: from remotemail by tesseract.cs.unb.ca with local (Exim 4.80)\r
  23         (envelope-from <bremner@unb.ca>)\r
  24         id 1ULkuT-00060l-Kp; Fri, 29 Mar 2013 22:48:45 -0300\r
  25 Received: (nullmailer pid 11278 invoked by uid 1000);\r
  26         Fri, 29 Mar 2013 23:05:54 -0000\r
  27 From: David Bremner <david@tethera.net>\r
  28 To: Jani Nikula <jani@nikula.org>, notmuch@notmuchmail.org\r
  29 Subject: Re: [PATCH 2/2] cli: mime node: abstract decryption and signature\r
  30         verification\r
  31 In-Reply-To:\r
  32  <d250faaac8aae1f6383b7a77e84e4b4144791688.1362319765.git.jani@nikula.org>\r
  33 References:\r
  34  <a9f12ba474fc51df71e2fd2b7a20a8d101729c6e.1362319765.git.jani@nikula.org>\r
  35         <d250faaac8aae1f6383b7a77e84e4b4144791688.1362319765.git.jani@nikula.org>\r
  36 User-Agent: Notmuch/0.15.1 (http://notmuchmail.org) Emacs/24.2.1\r
  37         (x86_64-pc-linux-gnu)\r
  38 Date: Fri, 29 Mar 2013 19:05:54 -0400\r
  39 Message-ID: <87k3opiz71.fsf@maritornes.cs.unb.ca>\r
  40 MIME-Version: 1.0\r
  41 Content-Type: text/plain\r
  42 X-BeenThere: notmuch@notmuchmail.org\r
  43 X-Mailman-Version: 2.1.13\r
  44 Precedence: list\r
  45 List-Id: "Use and development of the notmuch mail system."\r
  46         <notmuch.notmuchmail.org>\r
  47 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  48         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  49 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  50 List-Post: <mailto:notmuch@notmuchmail.org>\r
  51 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  52 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  53         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  54 X-List-Received-Date: Sat, 30 Mar 2013 01:48:53 -0000\r
  55 \r
  56 Jani Nikula <jani@nikula.org> writes:\r
  57 \r
  58 > The code filled with #ifdef GMIME_ATLEAST_26 is difficult to\r
  59 > read. Abstract the decryption and signature verification into\r
  60 > functions, with separate implementations for GMime 2.4 and 2.6, to\r
  61 > clarify the code.\r
  62 \r
  63 This series mostly looks OK, although it's a challenge to track all the\r
  64 code movement.\r
  65 \r
  66 I'd like some way to make it more obvious to the reader which version of\r
  67 the functions they are reading (when the #ifdef GMIME_ATLEAST_26 has\r
  68 scrolled off screen). The simplest would just be a comment in front of\r
  69 each function.\r
  70 \r
  71 d\r
  72 \r
  73 \r
  74 \r