Re: [PATCH] _notmuch_message_index_file: unref (free) address lists from gmime.
[notmuch-archives.git] / ab / 7de7a69beb781419d2a7b8d83f66a5be23431b
1 Return-Path: <sojkam1@fel.cvut.cz>\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 731DB40D14A\r
6         for <notmuch@notmuchmail.org>; Thu, 21 Oct 2010 21:43:43 -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: -1.9\r
10 X-Spam-Level: \r
11 X-Spam-Status: No, score=-1.9 tagged_above=-999 required=5\r
12         tests=[BAYES_00=-1.9] autolearn=ham\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 JakRnsHvU7mJ for <notmuch@notmuchmail.org>;\r
16         Thu, 21 Oct 2010 21:43:32 -0700 (PDT)\r
17 Received: from max.feld.cvut.cz (max.feld.cvut.cz [147.32.192.36])\r
18         by olra.theworths.org (Postfix) with ESMTP id 96B6D40D149\r
19         for <notmuch@notmuchmail.org>; Thu, 21 Oct 2010 21:43:32 -0700 (PDT)\r
20 Received: from localhost (unknown [192.168.200.4])\r
21         by max.feld.cvut.cz (Postfix) with ESMTP id 4636F19F33D8;\r
22         Fri, 22 Oct 2010 06:43:30 +0200 (CEST)\r
23 X-Virus-Scanned: IMAP AMAVIS\r
24 Received: from max.feld.cvut.cz ([192.168.200.1])\r
25         by localhost (styx.feld.cvut.cz [192.168.200.4]) (amavisd-new,\r
26         port 10044)\r
27         with ESMTP id 1pWza8y4uuJ1; Fri, 22 Oct 2010 06:43:29 +0200 (CEST)\r
28 Received: from imap.feld.cvut.cz (imap.feld.cvut.cz [147.32.192.34])\r
29         by max.feld.cvut.cz (Postfix) with ESMTP id ED95E19F33B8;\r
30         Fri, 22 Oct 2010 06:43:28 +0200 (CEST)\r
31 Received: from steelpick.2x.cz (note-sojka.felk.cvut.cz [147.32.86.30])\r
32         (Authenticated sender: sojkam1)\r
33         by imap.feld.cvut.cz (Postfix) with ESMTPSA id AEE3BFA006;\r
34         Fri, 22 Oct 2010 06:43:28 +0200 (CEST)\r
35 Received: from wsh by steelpick.2x.cz with local (Exim 4.72)\r
36         (envelope-from <sojkam1@fel.cvut.cz>)\r
37         id 1P99TU-0006yo-1k; Fri, 22 Oct 2010 06:43:28 +0200\r
38 From: Michal Sojka <sojkam1@fel.cvut.cz>\r
39 To: Mike Kelly <pioto@pioto.org>, notmuch@notmuchmail.org\r
40 Subject: Re: [PATCH 0/4] Maildir synchronization v2\r
41 In-Reply-To: <20101018101231.69904381@pioto.org>\r
42 References: <1286972038-21064-1-git-send-email-sojkam1@fel.cvut.cz>\r
43         <20101018101231.69904381@pioto.org>\r
44 User-Agent: Notmuch/0.3.1-110-g7be1648 (http://notmuchmail.org) Emacs/23.2.1\r
45         (x86_64-pc-linux-gnu)\r
46 Date: Fri, 22 Oct 2010 06:43:27 +0200\r
47 Message-ID: <8739ryltw0.fsf@steelpick.2x.cz>\r
48 MIME-Version: 1.0\r
49 Content-Type: text/plain; charset=us-ascii\r
50 X-BeenThere: notmuch@notmuchmail.org\r
51 X-Mailman-Version: 2.1.13\r
52 Precedence: list\r
53 List-Id: "Use and development of the notmuch mail system."\r
54         <notmuch.notmuchmail.org>\r
55 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
56         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
57 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
58 List-Post: <mailto:notmuch@notmuchmail.org>\r
59 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
60 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
61         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
62 X-List-Received-Date: Fri, 22 Oct 2010 04:43:43 -0000\r
63 \r
64 On Mon, 18 Oct 2010, Mike Kelly wrote:\r
65 > Michal Sojka <sojkam1@fel.cvut.cz> wrote:\r
66 > > they are ready to be merged. They can be pulled by:\r
67 > > \r
68 > > git pull git://rtime.felk.cvut.cz/notmuch maildir-sync-v2\r
69\r
70 > I've tested these patches on Linux, and they seem to work as expected.\r
71 > However, I'd like to suggest/request that the level-4 syncing of a\r
72 > filename should happen in notmuch_(message|thread)_(add|remove)_tag(),\r
73 > not in notmuch-tag.cc. Otherwise, anyone using libnotmuch directly\r
74 > (e.g. alternative clients, things using python bindings, etc) will not\r
75 > benefit from this.\r
76 \r
77 Hi, I'd agree but these pathes don't touch notmuch-tag.c in a\r
78 significant way. What's done there is to read the value from config and\r
79 to call a function (from libnotmuch) to setup an internal variable\r
80 accordingly. Other users of libnotmuch have probably different means of\r
81 configuration than notmuch itself and should call this function in a\r
82 similar way.\r
83 \r
84 -Michal\r