[prev in list] [next in list] [prev in thread] [next in thread] 

List:       zope-dev
Subject:    Re: [Zope-dev] Re: SVN: zope.component/ Merge
From:       Christian Theune <ct () gocept ! com>
Date:       2008-07-25 16:18:01
Message-ID: 1217002681.6904.94.camel () mindy
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Fri, 2008-07-25 at 16:58 +0100, Chris Withers wrote:
> Wichert Akkerman wrote:
> >> -        if (self._utility_registrations.get((provided, name))
> >> +        if (self._utility_registrations.get((provided, name)[:2]) 
> > 
> > Excellent catch, fixed.
> 
> With a test I hope?

Uh. Lol?

To me this looks like an inefficiency that's gonna be hard to test from
the outside. Or did I miss anything?

Christian

-- 
Christian Theune  · ct@gocept.com
gocept gmbh & co. kg  · forsterstraße 29  · 06112 halle (saale)  · germany
http://gocept.com  · tel +49 345 1229889 7  · fax +49 345 1229889 1
Zope and Plone consulting and development


["signature.asc" (application/pgp-signature)]

_______________________________________________
Zope-Dev maillist  -  Zope-Dev@zope.org
http://mail.zope.org/mailman/listinfo/zope-dev
**  No cross posts or HTML encoding!  **
(Related lists - 
 http://mail.zope.org/mailman/listinfo/zope-announce
 http://mail.zope.org/mailman/listinfo/zope )


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic