53/e8b35d725d4f67663edb15bd5bec6a0b8b2428

   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 DE1A34196F0\r
   6         for <notmuch@notmuchmail.org>; Thu,  1 Apr 2010 07:21:17 -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.001\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.001 tagged_above=-999 required=5\r
  12         tests=[BAYES_20=-0.001] 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 P+-NsDim6R0y for <notmuch@notmuchmail.org>;\r
  16         Thu,  1 Apr 2010 07:21:16 -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 C1060431FC1\r
  19         for <notmuch@notmuchmail.org>; Thu,  1 Apr 2010 07:21:16 -0700 (PDT)\r
  20 Received: from localhost (unknown [192.168.200.4])\r
  21         by max.feld.cvut.cz (Postfix) with ESMTP id 3029319F3614;\r
  22         Thu,  1 Apr 2010 16:21:16 +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 35eTLNqc6emK; Thu,  1 Apr 2010 16:21:15 +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 03CDA19F3361;\r
  30         Thu,  1 Apr 2010 16:21:14 +0200 (CEST)\r
  31 Received: from steelpick.2x.cz (k335-30.felk.cvut.cz [147.32.86.30])\r
  32         (Authenticated sender: sojkam1)\r
  33         by imap.feld.cvut.cz (Postfix) with ESMTPSA id C311AFA003;\r
  34         Thu,  1 Apr 2010 16:21:14 +0200 (CEST)\r
  35 Received: from wsh by steelpick.2x.cz with local (Exim 4.71)\r
  36         (envelope-from <sojkam1@fel.cvut.cz>)\r
  37         id 1NxLGk-0005hL-KI; Thu, 01 Apr 2010 16:21:14 +0200\r
  38 From: Michal Sojka <sojkam1@fel.cvut.cz>\r
  39 To: nstraz@redhat.com, notmuch@notmuchmail.org\r
  40 In-Reply-To: <20100401134721.GD25669@redhat.com>\r
  41 References: <20100401134721.GD25669@redhat.com>\r
  42 Date: Thu, 01 Apr 2010 16:21:14 +0200\r
  43 Message-ID: <87wrwr45k5.fsf@steelpick.2x.cz>\r
  44 MIME-Version: 1.0\r
  45 Content-Type: text/plain; charset=us-ascii\r
  46 Subject: Re: [notmuch] [PATCH] Setup the GMimeStream only when needed\r
  47 X-BeenThere: notmuch@notmuchmail.org\r
  48 X-Mailman-Version: 2.1.13\r
  49 Precedence: list\r
  50 List-Id: "Use and development of the notmuch mail system."\r
  51         <notmuch.notmuchmail.org>\r
  52 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  53         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  54 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  55 List-Post: <mailto:notmuch@notmuchmail.org>\r
  56 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  57 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  58         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  59 X-List-Received-Date: Thu, 01 Apr 2010 14:21:18 -0000\r
  60 \r
  61 On Thu, 01 Apr 2010, nstraz@redhat.com wrote:\r
  62 > I ran into this while looking at the vim plugin.  Vim's system() call\r
  63 > redirects output to a file and it was missing many of the part{ lines.\r
  64 > \r
  65 > If stream_stdout is setup too early, it will overwrite the part start\r
  66 > when notmuch is redirected to a file.\r
  67 \r
  68 Hi,\r
  69 \r
  70 thanks for the patch. After some investigation, it seems that you are\r
  71 right. What I missed in your commit message is the reason for such\r
  72 behaviour i.e. GMimeStream fseek()s in its write method to the position\r
  73 recorded when the stream was created, so that in case there is somebody\r
  74 else writing to the stream, the writes may overlap.\r
  75 \r
  76 -Michal\r