[Zope-dev] Merge proposal: tseaver-better_unittests branch of zope.interface

[Marius Gedminas]

On Mon, Mar 26, 2012 at 05:38:07PM -0400, Tres Seaver wrote:
> I've (finally!) finished my work to get zope.interface to 100% unit test
> coverage without relying on doctests:
> 
>   http://svn.zope.org/zope.interface/branches/tseaver-better_unittests/
> 
> The work is outlined in this document on the branch:
> 
> 
> http://svn.zope.org/zope.interface/branches/tseaver-better_unittests/README-better_unittest.txt?rev=124744&view=auto

I've a comment about this change, which was part of that large "merge
from launchpad" commit:

--- zope.interface/branches/tseaver-better_unittests/src/zope/interface/_zope_interface_coptimizations.c	(revision 118418)
+++ zope.interface/branches/tseaver-better_unittests/src/zope/interface/_zope_interface_coptimizations.c	(revision 124742)
@@ -980,5 +980,11 @@
     }
   else
-    Py_INCREF(result);
+    {
+      if (result == Py_None && default_ != NULL)
+        {
+          result = default_;
+        }
+      Py_INCREF(result);
+    }
 
   return result;

It seems to be a bugfix for http://pad.lv/910987 from [1]

[1] http://bazaar.launchpad.net/~tseaver/zope.interface/better_unittests/revision/182

I failed to find any mention of this CHANGES.txt on that branch.

Cheers!
Marius Gedminas
-- 
http://pov.lt/ -- Zope 3/BlueBream consulting and development
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://mail.zope.org/pipermail/zope-dev/attachments/20120328/5ee7681b/attachment.sig>