On Thu, Jul 24, 2008 at 1:44 PM, Wichert Akkerman <[EMAIL PROTECTED]> wrote:
> Log message for revision 88796:
>  Correct error in persistance-bbb logic spotted by Jacob Holm

It seems strange (and a bit frightening) that -- ostensibly -- the tests
pass either way the code reads.  Is there a missing test here?

> Changed:
>  U   zope.component/trunk/src/zope/component/registry.py
>
> -=-
> Modified: zope.component/trunk/src/zope/component/registry.py
> ===================================================================
> --- zope.component/trunk/src/zope/component/registry.py 2008-07-24 17:32:08 
> UTC (rev 88795)
> +++ zope.component/trunk/src/zope/component/registry.py 2008-07-24 17:44:53 
> UTC (rev 88796)
> @@ -75,7 +75,7 @@
>         if provided is None:
>             provided = _getUtilityProvided(component)
>
> -        if (self._utility_registrations.get((provided, name)[:2])
> +        if (self._utility_registrations.get((provided, name))[:2]
>             == (component, info)):
>             # already registered
>             return

-- 
Benji York
Senior Software Engineer
Zope Corporation
_______________________________________________
For more information about ZODB, see the ZODB Wiki:
http://www.zope.org/Wikis/ZODB/

ZODB-Dev mailing list  -  ZODB-Dev@zope.org
http://mail.zope.org/mailman/listinfo/zodb-dev

Reply via email to