From b7f56dfe4d9ed3f722ad3fe81bb5daa33466ed32 Mon Sep 17 00:00:00 2001
From: Bob van Loosen <bob.loosen@gmail.com>
Date: Tue, 26 Mar 2013 22:53:58 +0100
Subject: [PATCH] fixed: no need to lock if only checking for a closed socket

---
 src/bitvis/bitvis.cpp | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/bitvis/bitvis.cpp b/src/bitvis/bitvis.cpp
index e03a644..4b2282d 100644
--- a/src/bitvis/bitvis.cpp
+++ b/src/bitvis/bitvis.cpp
@@ -246,9 +246,9 @@ void CBitVis::Run()
     while ((msg = m_jackclient.GetMessage()) != MsgNone)
       LogDebug("got message %s from jack client", MsgToString(msg));
 
-    CLock lock(m_condition);
     if (!m_socket.IsOpen() && m_address && GetTimeUs() - lastconnect > CONNECTINTERVAL)
     {
+      CLock lock(m_condition);
       if (m_socket.Open(m_address, m_port, 10000000) != SUCCESS)
       {
         LogError("Failed to connect: %s", m_socket.GetError().c_str());
@@ -260,7 +260,6 @@ void CBitVis::Run()
       }
       didconnect = true;
     }
-    lock.Leave();
 
     if (didconnect)
       lastconnect = GetTimeUs();