Ticket #5016 enhancement closed fixed
Add a public API like twisted.internet.test.connectionmixins.serverFactoryFor
| Reported by: | exarkun | Owned by: | therve |
|---|---|---|---|
| Priority: | low | Milestone: | |
| Component: | core | Keywords: | |
| Cc: | Branch: | branches/factory-for-5016 | |
| Author: | therve | Launchpad Bug: |
Description
These two lines are simple but typing them out gets tedious:
factory = ServerFactory() factory.protocol = protocol
It would be nice to have this API provided by Twisted somewhere. Ideally it would just be ServerFactory(protocol), but ServerFactory has no __init__ and giving it one breaks this pattern:
class Foo: def __init__(self): self.stuff = things class Bar(ServerFactory, Foo): pass
(since suddenly Bar.__init__ means ServerFactory.__init__ instead of Foo.__init__).
Perhaps we could add a class method, though. ServerFactory.forProtocol(protocol, *args, **kwargs).
Maybe this belongs on Factory rather than ServerFactory, too.
Attachments
Change History
Note: See
TracTickets for help on using
tickets.

