f1/44e98993ec1b084fb5e5f12d0672cfbe7b5c24

   1 Return-Path: <jw+debian@jameswestby.net>\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 0D1F5429E38\r
   6         for <notmuch@notmuchmail.org>; Fri,  2 Dec 2011 06:20:45 -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: 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 G0VvFdVhzNCJ for <notmuch@notmuchmail.org>;\r
  16         Fri,  2 Dec 2011 06:20:41 -0800 (PST)\r
  17 Received: from jameswestby.net (jameswestby.net [89.145.97.141])\r
  18         (using TLSv1 with cipher AES256-SHA (256/256 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 2400C429E36\r
  21         for <notmuch@notmuchmail.org>; Fri,  2 Dec 2011 06:20:41 -0800 (PST)\r
  22 Received: from [74.220.184.116] (helo=dim.jameswestby.net)\r
  23         by jameswestby.net with esmtpa (Exim 4.69)\r
  24         (envelope-from <jw+debian@jameswestby.net>)\r
  25         id 1RWTyg-0007hx-1M; Fri, 02 Dec 2011 14:20:38 +0000\r
  26 Received: by dim.jameswestby.net (Postfix, from userid 1000)\r
  27         id 35E5F5A1D41; Fri,  2 Dec 2011 09:20:35 -0500 (EST)\r
  28 From: James Westby <jw+debian@jameswestby.net>\r
  29 To: Justus Winter <4winter@informatik.uni-hamburg.de>,\r
  30         Sebastian Spaeth <Sebastian@SSpaeth.de>, notmuch@notmuchmail.org\r
  31 Subject: Re: [PATCH 1/2] python: add classes to wrap all notmuch_*_t types\r
  32 In-Reply-To: <E1RWSKd-0001tZ-RN@thinkbox.jade-hamburg.de>\r
  33 References:\r
  34  <1318198374-926-1-git-send-email-4winter@informatik.uni-hamburg.de>\r
  35         <87vcr39e9q.fsf@SSpaeth.de>     <E1RV0Rw-00036z-O1@thinkbox.jade-hamburg.de>\r
  36         <87ty5kugq2.fsf@SSpaeth.de> <87hb1jc1gx.fsf@jameswestby.net>\r
  37         <E1RWSKd-0001tZ-RN@thinkbox.jade-hamburg.de>\r
  38 User-Agent: Notmuch/0.6.1-1 (http://notmuchmail.org) Emacs/23.3.1\r
  39         (x86_64-pc-linux-gnu)\r
  40 Date: Fri, 02 Dec 2011 09:20:35 -0500\r
  41 Message-ID: <87y5uvnjgs.fsf@jameswestby.net>\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: Fri, 02 Dec 2011 14:20:45 -0000\r
  57 \r
  58 On Fri, 02 Dec 2011 13:35:11 +0100, Justus Winter <4winter@informatik.uni-hamburg.de> wrote:\r
  59 > Huh, strange. My patch isn't supposed to change anything, it just\r
  60 > enables the ctypes package to check whether the functions from\r
  61 > libnotmuch are called with the right parameters, thus preventing\r
  62 > mistakes when changing the python bindings in the future.\r
  63 \r
  64 Where .restype is set to c_void_p ctypes spots this and returns it as\r
  65 int32. Where it's set to another callable (e.g. your custom types) it\r
  66 actually wraps the value by passing it to the callable and returning the\r
  67 result.\r
  68 \r
  69 Then, because your custom types are pointers, it stores them in an\r
  70 appropriate value, and also stops the storage being reused.\r
  71 \r
  72 I'll test again to make sure that I have this correct, but my tests\r
  73 yesterday certainly suggested that your patches fixed this.\r
  74 \r
  75 Thanks,\r
  76 \r
  77 James\r