[RFC PATCH 0/6] cli/reply: refactoring part 2
authorJani Nikula <jani@nikula.org>
Sun, 19 Jun 2016 20:15:28 +0000 (23:15 +0300)
committerW. Trevor King <wking@tremily.us>
Sat, 20 Aug 2016 23:22:07 +0000 (16:22 -0700)
6f/a7c6b9cb1cd906bf43270449053740264c0bdd [new file with mode: 0644]

diff --git a/6f/a7c6b9cb1cd906bf43270449053740264c0bdd b/6f/a7c6b9cb1cd906bf43270449053740264c0bdd
new file mode 100644 (file)
index 0000000..569c1a1
--- /dev/null
@@ -0,0 +1,107 @@
+Return-Path: <jani@nikula.org>\r
+X-Original-To: notmuch@notmuchmail.org\r
+Delivered-To: notmuch@notmuchmail.org\r
+Received: from localhost (localhost [127.0.0.1])\r
+ by arlo.cworth.org (Postfix) with ESMTP id 165D96DE0355\r
+ for <notmuch@notmuchmail.org>; Sun, 19 Jun 2016 13:17:11 -0700 (PDT)\r
+X-Virus-Scanned: Debian amavisd-new at cworth.org\r
+X-Spam-Flag: NO\r
+X-Spam-Score: -0.568\r
+X-Spam-Level: \r
+X-Spam-Status: No, score=-0.568 tagged_above=-999 required=5 tests=[AWL=0.152,\r
+  DKIM_SIGNED=0.1, DKIM_VALID=-0.1, RCVD_IN_DNSWL_LOW=-0.7,\r
+ RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01] autolearn=disabled\r
+Received: from arlo.cworth.org ([127.0.0.1])\r
+ by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024)\r
+ with ESMTP id sZL8uBU3xs3a for <notmuch@notmuchmail.org>;\r
+ Sun, 19 Jun 2016 13:17:03 -0700 (PDT)\r
+Received: from mail-wm0-f54.google.com (mail-wm0-f54.google.com\r
+ [74.125.82.54]) by arlo.cworth.org (Postfix) with ESMTPS id B3DD16DE00B8 for\r
+ <notmuch@notmuchmail.org>; Sun, 19 Jun 2016 13:17:02 -0700 (PDT)\r
+Received: by mail-wm0-f54.google.com with SMTP id r201so35834569wme.1\r
+ for <notmuch@notmuchmail.org>; Sun, 19 Jun 2016 13:17:02 -0700 (PDT)\r
+DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;\r
+ d=nikula-org.20150623.gappssmtp.com; s=20150623;\r
+ h=from:to:cc:subject:date:message-id:in-reply-to:references;\r
+ bh=gPyrvjGT2ojk74j/NJWxLEgByugDyYtW3610kx+48Kk=;\r
+ b=oE8M/ggnrBmz3A/3SFHmfQ5kQZqekFrJ0wVIw7LVq1wgDWOPzfjSmbmKip65YHLvjI\r
+ UJCNMYd1RuqS7M8dRQqeLQo/HHg7KWGvKRYLe2nGpvq0wVkziCg8OQmMfSWgHhJH0H3S\r
+ mBTng/hO9cCyIZJPAz6ELgh1Tgglu79avs9pwBeu7dGN2MEyii0f/mPkAXOoRHxZmBLC\r
+ pc9A5Ni/5kh/N9bjmxKCkk0wEY1AXxXayv5bYxS0/5knlvAt6bOfDqj85aHM9Su4fC7D\r
+ li57guhskj61O5fC4AbGL6UFM9zpHEuJrprflMdrcuK5kgMpfv0BHv+f08f4NrcemGvt\r
+ nuBg==\r
+X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;\r
+ d=1e100.net; s=20130820;\r
+ h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to\r
+ :references;\r
+ bh=gPyrvjGT2ojk74j/NJWxLEgByugDyYtW3610kx+48Kk=;\r
+ b=RL8yd+mTY49tWUQEknF7p65OdTlZk+7d1ojDfqm1g2IO5LP3FqVPSOU7KWKcgQJhXi\r
+ QKjnzxG8E8zNpZloqUiRPbZxzkK3bP2j1i5tcwfVe3LEQ+ckkQsk0K82496nJShSH+O/\r
+ Qw29bp5BBEUNLe1VsvYJxdWu8jAwd6bz+PatgoDBlFq/C/s25X5NbDlK56DOcOOhTgH+\r
+ 3mENZX0dPILiDteFQiYDedhrui8Afl9uME0OitJN/gXT1li/6ykCSiCvDth2s5CXGMw0\r
+ VdljWNQ2XH+FZqL+PKzhoNvcVyh+5wFiStIjHeee+s9ChXzp38yt9R2Zt/RPUxZfsq+C\r
+ XtwQ==\r
+X-Gm-Message-State:\r
+ ALyK8tLVcHCBZXFmA8bJvJjNfo99y+dYtbSfEfRF0I/h/eOipfjeQQpUmep50xZ8xx81kQ==\r
+X-Received: by 10.28.158.132 with SMTP id h126mr7423779wme.43.1466367420687;\r
+ Sun, 19 Jun 2016 13:17:00 -0700 (PDT)\r
+Received: from localhost (mobile-access-bcee4e-81.dhcp.inet.fi.\r
+ [188.238.78.81])\r
+ by smtp.gmail.com with ESMTPSA id ze10sm8031476wjb.2.2016.06.19.13.16.59\r
+ (version=TLSv1/SSLv3 cipher=OTHER);\r
+ Sun, 19 Jun 2016 13:17:00 -0700 (PDT)\r
+From: Jani Nikula <jani@nikula.org>\r
+To: Jani Nikula <jani@nikula.org>,\r
+       notmuch@notmuchmail.org\r
+Cc: Daniel Kahn Gillmor <dkg@fifthhorseman.net>\r
+Subject: [RFC PATCH 0/6] cli/reply: refactoring part 2\r
+Date: Sun, 19 Jun 2016 23:15:28 +0300\r
+Message-Id: <cover.1466366737.git.jani@nikula.org>\r
+X-Mailer: git-send-email 2.1.4\r
+In-Reply-To: <cover.1466284726.git.jani@nikula.org>\r
+References: <cover.1466284726.git.jani@nikula.org>\r
+X-BeenThere: notmuch@notmuchmail.org\r
+X-Mailman-Version: 2.1.20\r
+Precedence: list\r
+List-Id: "Use and development of the notmuch mail system."\r
+ <notmuch.notmuchmail.org>\r
+List-Unsubscribe: <https://notmuchmail.org/mailman/options/notmuch>,\r
+ <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
+List-Archive: <http://notmuchmail.org/pipermail/notmuch/>\r
+List-Post: <mailto:notmuch@notmuchmail.org>\r
+List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
+List-Subscribe: <https://notmuchmail.org/mailman/listinfo/notmuch>,\r
+ <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
+X-List-Received-Date: Sun, 19 Jun 2016 20:17:11 -0000\r
+\r
+This one expands on [1] to use GMimeMessage more. There are some less\r
+than perfect intermediate steps in order to split the stuff into\r
+reasonable sized patches, but it gets better in the end. There's still\r
+more to do, but I ran out of steam a bit (thus RFC), and it's better to\r
+get the first series merged first anyway.\r
+\r
+This should fix the issue Daniel saw [2], but I haven't really tested\r
+that, as there are no dedicated tests for it. Daniel, care to try this\r
+on the problematic message? Reindexing is not needed.\r
+\r
+BR,\r
+Jani.\r
+\r
+[1] id:cover.1466284726.git.jani@nikula.org\r
+[2] id:87d1ngv95p.fsf@alice.fifthhorseman.net\r
+\r
+\r
+Jani Nikula (6):\r
+  cli/reply: use dedicated functions for reply to mapping\r
+  cli/reply: check for NULL list first in scan_address_list()\r
+  cli/reply: return internet address list from get header funcs\r
+  cli/reply: pass internet address list to munge detect\r
+  cli/reply: pass gmime message to munge detection\r
+  cli/reply: only pass gmime message to add recipients to reply message\r
+\r
+ notmuch-reply.c | 171 +++++++++++++++++++++++++++++---------------------------\r
+ 1 file changed, 90 insertions(+), 81 deletions(-)\r
+\r
+-- \r
+2.1.4\r
+\r