-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Chris McDonough wrote:
> Author: Chris McDonough <chr...@agendaless.com>
> Date: Wed Jan 14 22:16:18 2009
> New Revision: 3174
> 
> Log:
> Fix interfaces.
> 
> 
> Modified:
>    repoze.bfg/trunk/repoze/bfg/interfaces.py
> 
> Modified: repoze.bfg/trunk/repoze/bfg/interfaces.py
> ==============================================================================
> --- repoze.bfg/trunk/repoze/bfg/interfaces.py (original)
> +++ repoze.bfg/trunk/repoze/bfg/interfaces.py Wed Jan 14 22:16:18 2009
> @@ -30,7 +30,7 @@
>  
>  class IRequestFactory(Interface):
>      """ A utility which generates a request factory """
> -    def __call__(self):
> +    def __call__(environ):
>          """ Return a request factory (e.g. a callable that accepts an
>          environ and returns an object implementing IRequest,
>          e.g. ``webob.Request``)"""

Doesn't IRequestFactory take an environ and give you back a request?  Or
is this the HammerFactoryFactory you were joking about on Tuesday?  If
it is a factory-factory, then why does it need access to the environ?

> @@ -55,7 +55,7 @@
>      
>  class IResponseFactory(Interface):
>      """ A utility which generates a response factory """
> -    def __call__(self):
> +    def __call__(*arg, **kw):
>          """ Return a response factory (e.g. a callable that returns an
>          object implementing IResponse, e.g. ``webob.Response``; it
>          should accept all the arguments that the webob.Response class



Tres.
- --
===================================================================
Tres Seaver          +1 540-429-0999          tsea...@palladion.com
Palladion Software   "Excellence by Design"    http://palladion.com
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFJbqtq+gerLs4ltQ4RAuCqAJ494DnrrenD4olZwRrv8hjc7SlpswCgkHYA
b5jhBbWAm7N7Nu/zY0r1Gcg=
=S6JV
-----END PGP SIGNATURE-----
_______________________________________________
Repoze-dev mailing list
Repoze-dev@lists.repoze.org
http://lists.repoze.org/listinfo/repoze-dev

Reply via email to