Code review comment for lp:~dne/txamqp/auto-channel-id-alloc

Revision history for this message
Daniel NĂ©ri (dne) wrote :

Thanks; I'll have a look at adding tests and docstrings.

I've also realised that IDs of closed channels should be reused, to avoid that self.channels fills up with "closed" AMQPChannel instances.

« Back to merge proposal