[Zope-dev] Re: SVN: zope.component/ Merge wichert-utility-factories branch to trunk

Wichert Akkerman wichert at wiggy.net
Fri Jul 25 18:00:24 EDT 2008


Christian Theune wrote:
> 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?
>    

No.

Wichert.

-- 
Wichert Akkerman<wichert at wiggy.net>    It is simple to make things.
http://www.wiggy.net/                  It is hard to make things simple.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.zope.org/pipermail/zope-dev/attachments/20080726/3f20d976/attachment.html


More information about the Zope-Dev mailing list