[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

Reply via email to