dvb: cosmetic
authorMirakels <mirakels@openpli.org>
Sat, 26 Dec 2015 11:04:40 +0000 (12:04 +0100)
committerMirakels <mirakels@openpli.org>
Sat, 26 Dec 2015 11:04:40 +0000 (12:04 +0100)
lib/dvb/dvb.cpp

index c1ca2a2..281d95f 100644 (file)
@@ -858,24 +858,21 @@ alloc_fe_by_id_not_possible:
 
 RESULT eDVBResourceManager::allocateDemux(eDVBRegisteredFrontend *fe, ePtr<eDVBAllocatedDemux> &demux, int &cap)
 {
-       /* find first unused demux which is on same adapter as frontend (or any, if PVR)
-        * never use the first one unless we need a decoding demux.
-        * look for a demux on the same adapter as the frontend, or the first adapter for dvr playback
-        */
-       iDVBAdapter *adapter = fe ? fe->m_adapter : m_adapter.begin();
-       int fesource = fe ? fe->m_frontend->getDVBID() : -1;
-
        eDebug("[eDVBResourceManager] allocate demux cap=%02X", cap);
        eSmartPtrList<eDVBRegisteredDemux>::iterator i(m_demux.begin());
 
        if (i == m_demux.end())
                return -1;
 
+       iDVBAdapter *adapter = fe ? fe->m_adapter : m_adapter.begin();
+       int fesource = fe ? fe->m_frontend->getDVBID() : -1;
+       ePtr<eDVBRegisteredDemux> unused;
+       uint8_t d, a;
+
        /*
         * For pvr playback, start with the last demux.
-        * On some hardware, we have less ca devices than demuxes,
-        * so we should try to leave the first demuxes for live tv,
-        * and start with the last for pvr playback
+        * On some hardware, there are less ca devices than demuxes, so try to leave
+        * the first demuxes for live tv, and start with the last for pvr playback
         */
        bool use_decode_demux = (fe || (cap & iDVBChannel::capDecode));
 
@@ -885,21 +882,19 @@ RESULT eDVBResourceManager::allocateDemux(eDVBRegisteredFrontend *fe, ePtr<eDVBA
                --i;
        }
 
-       ePtr<eDVBRegisteredDemux> unused;
-       uint8_t d, a;
-
        while (i != m_demux.end())
        {
                if (i->m_adapter == adapter)
                {
                        if (!i->m_inuse)
                        {
-                               /* mark the first unused demux, we'll use that when we do not find a better match */
-                               if (!unused) unused = i;
+                               // mark the first unused demux and use that when no better match is found
+                               if (!unused)
+                                       unused = i;
                        }
                        else
                        {
-                               /* demux is in use, see if we can share it */
+                               // demux is in use, see if it can be shared
                                if (fesource >= 0 && i->m_demux->getSource() == fesource)
                                {
                                        i->m_demux->getCAAdapterID(a);
@@ -916,7 +911,7 @@ RESULT eDVBResourceManager::allocateDemux(eDVBRegisteredFrontend *fe, ePtr<eDVBA
                }
                else
                {
-                       if(i == m_demux.begin())
+                       if (i == m_demux.begin())
                                break;
                        --i;
                }