OrchidSocketImpl: don't hold the stream lock whilst calling into stream.

I tried to minimise whitespace changes but tabs are a pain ... perhaps we should just untabify the whole codebase. It can always be retabified later if we decide to merge back with upstream.
Comparing to the old code, there is a race condition now where two threads could create two streams and assign it to the stream field.  That will cause one of the streams to be dangling.

The questions is whether this is a significant issue and whether the contract for this class disallows concurrent calls to connect().

If we decide this is significant, we could wrap this function in an additional lock.