e0/96d9c9a2eed980f8d5fcee775a4c48b29ad62f

   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 4A2AE429E25\r
   6         for <notmuch@notmuchmail.org>; Sat, 19 Nov 2011 04:59:36 -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: -2.3\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.3 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_MED=-2.3] 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 AFgZBbaGFVEa for <notmuch@notmuchmail.org>;\r
  16         Sat, 19 Nov 2011 04:59:35 -0800 (PST)\r
  17 Received: from tempo.its.unb.ca (tempo.its.unb.ca [131.202.1.21])\r
  18         (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id B53F8431FD0\r
  21         for <notmuch@notmuchmail.org>; Sat, 19 Nov 2011 04:59:35 -0800 (PST)\r
  22 Received: from zancas.localnet\r
  23         (fctnnbsc36w-156034074106.pppoe-dynamic.High-Speed.nb.bellaliant.net\r
  24         [156.34.74.106]) (authenticated bits=0)\r
  25         by tempo.its.unb.ca (8.13.8/8.13.8) with ESMTP id pAJCxUTA008951\r
  26         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NO);\r
  27         Sat, 19 Nov 2011 08:59:31 -0400\r
  28 Received: from bremner by zancas.localnet with local (Exim 4.77)\r
  29         (envelope-from <bremner@unb.ca>)\r
  30         id 1RRkW1-0005VA-Vn; Sat, 19 Nov 2011 08:59:29 -0400\r
  31 From: David Bremner <david@tethera.net>\r
  32 To: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>, notmuch@notmuchmail.org\r
  33 Subject: Re: [PATCH v2] Output unmodified Content-Type header value for JSON\r
  34         format.\r
  35 In-Reply-To: <1321676321-27745-1-git-send-email-dmitry.kurochkin@gmail.com>\r
  36 References: <1321659905-24367-1-git-send-email-dmitry.kurochkin@gmail.com>\r
  37         <1321676321-27745-1-git-send-email-dmitry.kurochkin@gmail.com>\r
  38 User-Agent: Notmuch/0.10~rc2 (http://notmuchmail.org) Emacs/23.3.1\r
  39         (x86_64-pc-linux-gnu)\r
  40 Date: Sat, 19 Nov 2011 08:59:29 -0400\r
  41 Message-ID: <87sjlktgi6.fsf@zancas.localnet>\r
  42 MIME-Version: 1.0\r
  43 Content-Type: text/plain; charset=us-ascii\r
  44 X-BeenThere: notmuch@notmuchmail.org\r
  45 X-Mailman-Version: 2.1.13\r
  46 Precedence: list\r
  47 List-Id: "Use and development of the notmuch mail system."\r
  48         <notmuch.notmuchmail.org>\r
  49 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  50         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  51 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  52 List-Post: <mailto:notmuch@notmuchmail.org>\r
  53 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  54 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  55         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  56 X-List-Received-Date: Sat, 19 Nov 2011 12:59:36 -0000\r
  57 \r
  58 On Sat, 19 Nov 2011 08:18:41 +0400, Dmitry Kurochkin <dmitry.kurochkin@gmail.com> wrote:\r
  59 > Before the change, notmuch used g_mime_content_type_to_string(3)\r
  60 > function to output Content-Type header value.  Turns out it outputs\r
  61 > only "type/subtype" part and ignores all parameters.  Also, if there\r
  62 > is no Content-Type header, default "text/plain" value is used.\r
  63 \r
  64 Hi Dmitry;\r
  65 \r
  66 I haven't analyzed the substance of your patch yet, but I did have a\r
  67 couple thoughts while reading your mail.\r
  68 \r
  69 - It seems that every time we change the json format, we have a round of\r
  70   suffering because people are unable to detect a mismatch between their\r
  71   emacs code and the cli. Not that this is your problem necessarily, but\r
  72   it would be nice if someone (TM), would come up with some version info\r
  73   for the json output, and a patch to check it on the emacs side.\r
  74 \r
  75 - The previous point is a bit of a counterargument to this, but in\r
  76   general, I think I prefer patches that modify the core seperate from\r
  77   those that do emacs (or python, or ...) stuff.\r
  78 \r
  79 - I understand you want to make your patches reviewable without applying\r
  80   by including lots of context, but at a certain point it has actually\r
  81   the opposite effect for me. I just don't read 900+ line emails ;). Of\r
  82   course, I can still apply the patch and look at it, so it's really up\r
  83   to you.\r
  84 \r
  85 d\r