[email protected] wrote:
> http://cr.opensolaris.org/~johansen/webrev-11854/
General high-level comment:
This might be better structured with a synchronization decorator, such
as the one demonstrated at
http://code.activestate.com/recipes/465057/
That would allow you to not have to separate all these methods out into
two quite as conspicuously. Now, that alone may not work, since the
Lock objects aren't recursive, so you'd probably have to switch to an
RLock object, which I think is safe, because you're only trying to
prevent different threads from using the transport object
simultaneously.
As is, though, it looks like it should correct the problem.
Danek
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss