Correct JackPosixThread::ThreadHandler termination, do not set buffer size if same value is used.

git-svn-id: http://subversion.jackaudio.org/jack/jack2/trunk/jackmp@2519 0c269be4-1314-0410-8aa9-9f06e86f4224

ChangeLog

@@ -21,6 +21,10 @@ Romain Moret
   Jackdmp changes log
 ---------------------------
 
+2008-06-13 Stephane Letz  <letz@grame.fr>
+
+	* Correct JackPosixThread::ThreadHandler termination, do not set buffer size if same value is used.
+
 2008-06-12 Stephane Letz  <letz@grame.fr>
 
 	* Another Tim Blechmann patch to remove unnecessary virtual methods.

common/JackPosixThread.cpp

@@ -59,7 +59,8 @@ void* JackPosixThread::ThreadHandler(void* arg)
     }
 
     jack_log("ThreadHandler: exit");
-    return 0;
+    pthread_exit(0);
+    return 0; // never reached
 }
 
 int JackPosixThread::Start()

common/JackServer.cpp

@@ -189,6 +189,11 @@ int JackServer::SetBufferSize(jack_nframes_t buffer_size)
     jack_log("JackServer::SetBufferSize nframes = %ld", buffer_size);
     jack_nframes_t current_buffer_size = fEngineControl->fBufferSize;
 
+    if (current_buffer_size == buffer_size) {
+        jack_log("SetBufferSize: requirement for new buffer size equals current value");
+        return 0;
+    }
+
     if (fAudioDriver->Stop() != 0) {
         jack_error("Cannot stop audio driver");
         return -1;

common/JackThreadedDriver.cpp

@@ -54,7 +54,7 @@ int JackThreadedDriver::Start()
         jack_error("Cannot start driver");
         return -1;
     }
-    if (fThread.Start() < 0) {
+    if (fThread.StartSync() < 0) {
         jack_error("Cannot start thread");
         return -1;
     }