Ticket #11194: 11194-v1.patch

mythtv/libs/libmythtv/AirPlay/mythairplayserver.cpp

@@ -22 +22 @@
 #include "bonjourregister.h"
 #include "mythairplayserver.h"
 
-MythAirplayServer* MythAirplayServer::gMythAirplayServer = NULL;
-MThread*           MythAirplayServer::gMythAirplayServerThread = NULL;
-QMutex*            MythAirplayServer::gMythAirplayServerMutex = new QMutex(QMutex::Recursive);
+static MythAirplayServer* s_airplayServer = NULL;
+static MThread*           s_airplayServerThread = NULL;
+static QMutex             s_airplayServerMutex;
 
 #define LOC QString("AirPlay: ")
 
@@ -307 +307 @@
 
 bool MythAirplayServer::Create(void)
 {
-    QMutexLocker locker(gMythAirplayServerMutex);
+    QMutexLocker locker(&s_airplayServerMutex);
 
     // create the server thread
-    if (!gMythAirplayServerThread)
-        gMythAirplayServerThread = new MThread("AirplayServer");
-    if (!gMythAirplayServerThread)
+    if (!s_airplayServerThread)
+        s_airplayServerThread = new MThread("AirplayServer");
+    if (!s_airplayServerThread)
     {
         LOG(VB_GENERAL, LOG_ERR, LOC + "Failed to create airplay thread.");
         return false;
     }
 
     // create the server object
-    if (!gMythAirplayServer)
-        gMythAirplayServer = new MythAirplayServer();
-    if (!gMythAirplayServer)
+    if (!s_airplayServer)
+        s_airplayServer = new MythAirplayServer();
+    if (!s_airplayServer)
     {
         LOG(VB_GENERAL, LOG_ERR, LOC + "Failed to create airplay object.");
+        delete s_airplayServerThread;
+        s_airplayServerThread = NULL;
         return false;
     }
 
     // start the thread
-    if (!gMythAirplayServerThread->isRunning())
+    if (!s_airplayServerThread->isRunning())
     {
-        gMythAirplayServer->moveToThread(gMythAirplayServerThread->qthread());
+        s_airplayServer->moveToThread(s_airplayServerThread->qthread());
         QObject::connect(
-            gMythAirplayServerThread->qthread(), SIGNAL(started()),
-            gMythAirplayServer,                  SLOT(Start()));
-        gMythAirplayServerThread->start(QThread::LowestPriority);
+            s_airplayServerThread->qthread(), SIGNAL(started()),
+            s_airplayServer,                  SLOT(Start()));
+        s_airplayServerThread->start(QThread::LowestPriority);
     }
 
     LOG(VB_GENERAL, LOG_INFO, LOC + "Created airplay objects.");
@@ -345 +347 @@
 {
     LOG(VB_GENERAL, LOG_INFO, LOC + "Cleaning up.");
 
-    if (gMythAirplayServer)
-        gMythAirplayServer->Teardown();
+    QMutexLocker locker(&s_airplayServerMutex);
 
-    QMutexLocker locker(gMythAirplayServerMutex);
-    if (gMythAirplayServerThread)
+    if (s_airplayServer)
+        s_airplayServer->Teardown();
+
+    if (s_airplayServerThread)
     {
-        gMythAirplayServerThread->exit();
-        gMythAirplayServerThread->wait();
+        s_airplayServerThread->exit();
+        s_airplayServerThread->wait();
     }
-    delete gMythAirplayServerThread;
-    gMythAirplayServerThread = NULL;
 
-    delete gMythAirplayServer;
-    gMythAirplayServer = NULL;
+    delete s_airplayServer;
+    s_airplayServer = NULL;
+
+    delete s_airplayServerThread;
+    s_airplayServerThread = NULL;
 }
 
 

mythtv/libs/libmythtv/AirPlay/mythairplayserver.h

@@ -59 +59 @@
   public:
     static bool Create(void);
     static void Cleanup(void);
-    static MythAirplayServer *AirplaySharedInstance(void)
-    { return gMythAirplayServer; }
 
     MythAirplayServer();
 
@@ -94 +92 @@
     void PausePlayback(void);
     void UnpausePlayback(void);
 
-    // Globals
-    static MythAirplayServer *gMythAirplayServer;
-    static QMutex            *gMythAirplayServerMutex;
-    static MThread           *gMythAirplayServerThread;
-
     // Members
     QString          m_name;
     BonjourRegister *m_bonjour;