[Cores] Performance: pre-increment/decrement instead of post-increment/decrement.
authorace20022 <ace20022@ymail.com>
Wed, 13 Mar 2013 14:06:09 +0000 (15:06 +0100)
committerace20022 <ace20022@ymail.com>
Fri, 5 Apr 2013 08:32:57 +0000 (10:32 +0200)
17 files changed:
xbmc/cores/DllLoader/dll_tracker.cpp
xbmc/cores/VideoRenderers/OverlayRenderer.cpp
xbmc/cores/VideoRenderers/RenderManager.cpp
xbmc/cores/amlplayer/AMLPlayer.cpp
xbmc/cores/dvdplayer/DVDCodecs/Video/DVDVideoCodecFFmpeg.cpp
xbmc/cores/dvdplayer/DVDCodecs/Video/DXVA.cpp
xbmc/cores/dvdplayer/DVDCodecs/Video/VAAPI.cpp
xbmc/cores/dvdplayer/DVDInputStreams/DVDInputStreamBluray.cpp
xbmc/cores/dvdplayer/DVDInputStreams/DVDInputStreamHTSP.cpp
xbmc/cores/dvdplayer/DVDInputStreams/DVDInputStreamStack.cpp
xbmc/cores/dvdplayer/DVDMessageQueue.cpp
xbmc/cores/dvdplayer/DVDMessageTracker.cpp
xbmc/cores/dvdplayer/DVDOverlayContainer.cpp
xbmc/cores/dvdplayer/DVDPlayerAudio.cpp
xbmc/cores/dvdplayer/DVDPlayerSubtitle.cpp
xbmc/cores/dvdplayer/Edl.cpp
xbmc/cores/playercorefactory/PlayerCoreFactory.cpp

index 53054f9..c40dfd5 100644 (file)
@@ -65,14 +65,14 @@ void tracker_dll_free(DllLoader* pDll)
       }
       // free all functions which where created at the time we loaded the dll
            DummyListIter dit = (*it)->dummyList.begin();
-           while (dit != (*it)->dummyList.end()) { free((void*)*dit); dit++;   }
+           while (dit != (*it)->dummyList.end()) { free((void*)*dit); ++dit;   }
            (*it)->dummyList.clear();
        
       delete (*it);
       it = g_trackedDlls.erase(it);
     }
     else
-      it++;
+      ++it;
   }
 }
 
index b00f28e..4dd27b7 100644 (file)
@@ -130,7 +130,7 @@ void CRenderer::Release(SElementV& list)
   SElementV l = list;
   list.clear();
 
-  for(SElementV::iterator it = l.begin(); it != l.end(); it++)
+  for(SElementV::iterator it = l.begin(); it != l.end(); ++it)
   {
     if(it->overlay)
       it->overlay->Release();
@@ -144,7 +144,7 @@ void CRenderer::Release(COverlayV& list)
   COverlayV l = list;
   list.clear();
 
-  for(COverlayV::iterator it = l.begin(); it != l.end(); it++)
+  for(COverlayV::iterator it = l.begin(); it != l.end(); ++it)
     (*it)->Release();
 }
 
@@ -175,7 +175,7 @@ void CRenderer::Render()
   Release(m_cleanup);
 
   SElementV& list = m_buffers[m_render];
-  for(SElementV::iterator it = list.begin(); it != list.end(); it++)
+  for(SElementV::iterator it = list.begin(); it != list.end(); ++it)
   {
     COverlay* o = NULL;
 
index ba57d49..c7ec3f9 100644 (file)
@@ -501,7 +501,7 @@ void CXBMCRenderManager::ManageCaptures()
         if (capture->IsAsync() && !(capture->GetFlags() & CAPTUREFLAG_IMMEDIATELY))
           RenderCapture(capture);
 
-        it++;
+        ++it;
       }
       else
       {
@@ -510,7 +510,7 @@ void CXBMCRenderManager::ManageCaptures()
     }
     else
     {
-      it++;
+      ++it;
     }
   }
 
index 705a547..c97b861 100644 (file)
@@ -327,7 +327,7 @@ void CAMLSubTitleThread::UpdateSubtitle(CStdString &subtitle, int64_t elapsed_ms
       if (elapsed_ms > amlsubtitle->endtime)
         it = m_subtitle_strings.erase(it);
       else
-        it++;
+        ++it;
     }
 
     // find the current subtitle
@@ -340,7 +340,7 @@ void CAMLSubTitleThread::UpdateSubtitle(CStdString &subtitle, int64_t elapsed_ms
         subtitle = amlsubtitle->string;
         break;
       }
-      it++;
+      ++it;
     }
   }
 }
index 9433f9b..4804b1b 100644 (file)
@@ -294,7 +294,7 @@ bool CDVDVideoCodecFFmpeg::Open(CDVDStreamInfo &hints, CDVDCodecOptions &options
   }
 
   // set any special options
-  for(std::vector<CDVDCodecOption>::iterator it = options.m_keys.begin(); it != options.m_keys.end(); it++)
+  for(std::vector<CDVDCodecOption>::iterator it = options.m_keys.begin(); it != options.m_keys.end(); ++it)
   {
     if (it->m_name == "surfaces")
       m_uSurfacesCount = std::atoi(it->m_value.c_str());
index 466a03a..257c096 100644 (file)
@@ -313,7 +313,7 @@ CSurfaceContext::CSurfaceContext()
 
 CSurfaceContext::~CSurfaceContext()
 {
-  for (vector<IDirect3DSurface9*>::iterator it = m_heldsurfaces.begin(); it != m_heldsurfaces.end(); it++)
+  for (vector<IDirect3DSurface9*>::iterator it = m_heldsurfaces.begin(); it != m_heldsurfaces.end(); ++it)
     SAFE_RELEASE(*it);
 }
 
@@ -1490,7 +1490,7 @@ bool CProcessor::Render(CRect src, CRect dst, IDirect3DSurface9* target, REFEREN
       it = m_sample.erase(it);
     }
     else
-      it++;
+      ++it;
   }
 
   if(m_sample.empty())
@@ -1520,7 +1520,7 @@ bool CProcessor::Render(CRect src, CRect dst, IDirect3DSurface9* target, REFEREN
   for (int i = 0; i < count; i++)
     samp[i].SampleFormat.SampleFormat = DXVA2_SampleUnknown;
 
-  for(it = m_sample.begin(); it != m_sample.end() && valid < count; it++)
+  for(it = m_sample.begin(); it != m_sample.end() && valid < count; ++it)
   {
     if (it->sample.Start >= MinTime && it->sample.Start <= MaxTime)
     {
index 27903e3..5d10f31 100644 (file)
@@ -161,7 +161,7 @@ void CDecoder::RelBuffer(AVCodecContext *avctx, AVFrame *pic)
 {
   VASurfaceID surface = GetSurfaceID(pic);
 
-  for(std::list<CSurfacePtr>::iterator it = m_surfaces_used.begin(); it != m_surfaces_used.end(); it++)
+  for(std::list<CSurfacePtr>::iterator it = m_surfaces_used.begin(); it != m_surfaces_used.end(); ++it)
   {    
     if((*it)->m_id == surface)
     {
@@ -184,7 +184,7 @@ int CDecoder::GetBuffer(AVCodecContext *avctx, AVFrame *pic)
   if(surface)
   {
     /* reget call */
-    for(; it != m_surfaces_free.end(); it++)
+    for(; it != m_surfaces_free.end(); ++it)
     {
       if((*it)->m_id == surface)
       {
@@ -204,7 +204,7 @@ int CDecoder::GetBuffer(AVCodecContext *avctx, AVFrame *pic)
   {
     // To avoid stutter, we scan the free surface pool (provided by decoder) for surfaces
     // that are 100% not in use by renderer. The pointers to these surfaces have a use_count of 1.
-    for (; it != m_surfaces_free.end() && it->use_count() > 1; it++) {}
+    for (; it != m_surfaces_free.end() && it->use_count() > 1; ++it) {}
 
     // If we have zero free surface from decoder OR all free surfaces are in use by renderer, we allocate a new surface
     if (it == m_surfaces_free.end())
@@ -217,7 +217,7 @@ int CDecoder::GetBuffer(AVCodecContext *avctx, AVFrame *pic)
         return -1;
       }
       // Set itarator position to the newly allocated surface (end-1)
-      it = m_surfaces_free.end(); it--;
+      it = m_surfaces_free.end(); --it;
     }
     /* getbuffer call */
     wrapper = it->get();
@@ -452,7 +452,7 @@ bool CDecoder::GetPicture(AVCodecContext* avctx, AVFrame* frame, DVDVideoPicture
   m_holder.surface.reset();
 
   std::list<CSurfacePtr>::iterator it;
-  for(it = m_surfaces_used.begin(); it != m_surfaces_used.end() && !m_holder.surface; it++)
+  for(it = m_surfaces_used.begin(); it != m_surfaces_used.end() && !m_holder.surface; ++it)
   {    
     if((*it)->m_id == surface)
     {
@@ -461,7 +461,7 @@ bool CDecoder::GetPicture(AVCodecContext* avctx, AVFrame* frame, DVDVideoPicture
     }
   }
 
-  for(it = m_surfaces_free.begin(); it != m_surfaces_free.end() && !m_holder.surface; it++)
+  for(it = m_surfaces_free.begin(); it != m_surfaces_free.end() && !m_holder.surface; ++it)
   {    
     if((*it)->m_id == surface)
     {
index ea5493e..58b2968 100644 (file)
@@ -693,7 +693,7 @@ void CDVDInputStreamBluray::OverlayClear(SPlane& plane, int x, int y, int w, int
     /* if no overlap we are done */
     if(rem.size() == 1 && !(rem[0] != old))
     {
-      it++;
+      ++it;
       continue;
     }
 
index 6c8fc7c..91e86d1 100644 (file)
@@ -181,14 +181,14 @@ bool CDVDInputStreamHTSP::SetChannel(int channel)
 
 bool CDVDInputStreamHTSP::GetChannels(SChannelV &channels, SChannelV::iterator &it)
 {
-  for(SChannels::iterator it2 = m_channels.begin(); it2 != m_channels.end(); it2++)
+  for(SChannels::iterator it2 = m_channels.begin(); it2 != m_channels.end(); ++it2)
   {
     if(m_tag == 0 || it2->second.MemberOf(m_tag))
       channels.push_back(it2->second);
   }
   sort(channels.begin(), channels.end());
 
-  for(it = channels.begin(); it != channels.end(); it++)
+  for(it = channels.begin(); it != channels.end(); ++it)
     if(it->id == m_channel)
       return true;
   return false;
index 9e9ec0f..a8e5e00 100644 (file)
@@ -147,7 +147,7 @@ int64_t CDVDInputStreamStack::Seek(int64_t offset, int whence)
     return -1;
 
   len = 0;
-  for(TSegVec::iterator it = m_files.begin(); it != m_files.end(); it++)
+  for(TSegVec::iterator it = m_files.begin(); it != m_files.end(); ++it)
   {
     if(len + it->length > pos)
     {
index 7d02cd5..b60d509 100644 (file)
@@ -67,7 +67,7 @@ void CDVDMessageQueue::Flush(CDVDMsg::Message type)
     if (it->message->IsType(type) ||  type == CDVDMsg::NONE)
       it = m_list.erase(it);
     else
-      it++;
+      ++it;
   }
 
   if (type == CDVDMsg::DEMUXER_PACKET ||  type == CDVDMsg::NONE)
@@ -121,7 +121,7 @@ MsgQueueReturnCode CDVDMessageQueue::Put(CDVDMsg* pMsg, int priority)
   {
     if(priority <= it->priority)
       break;
-    it++;
+    ++it;
   }
   m_list.insert(it, DVDMessageListItem(pMsg, priority));
 
@@ -230,7 +230,7 @@ unsigned CDVDMessageQueue::GetPacketCount(CDVDMsg::Message type)
     return 0;
 
   unsigned count = 0;
-  for(SList::iterator it = m_list.begin(); it != m_list.end();it++)
+  for(SList::iterator it = m_list.begin(); it != m_list.end();++it)
   {
     if(it->message->IsType(type))
       count++;
index 5741b94..32ad76c 100644 (file)
@@ -76,7 +76,7 @@ void CDVDMessageTracker::UnRegister(CDVDMsg* pMsg)
       delete pItem;
       break;
     }
-    iter++;
+    ++iter;
   }
 
 }
@@ -108,7 +108,7 @@ void CDVDMessageTracker::Process()
           pItem->m_debug_logged = true;
         }
       }
-      iter++;
+      ++iter;
     }
 
   }
index 50c3e67..cdc4b84 100644 (file)
@@ -121,7 +121,7 @@ void CDVDOverlayContainer::CleanUp(double pts)
         continue;
       }
     }
-    it++;
+    ++it;
   }
 
 }
@@ -162,7 +162,7 @@ bool CDVDOverlayContainer::ContainsOverlayType(DVDOverlayType type)
   while (!result && it != m_overlays.end())
   {
     if (((CDVDOverlay*)*it)->IsOverlayType(type)) result = true;
-    it++;
+    ++it;
   }
 
   return result;
@@ -176,7 +176,7 @@ void CDVDOverlayContainer::UpdateOverlayInfo(CDVDInputStreamNavigator* pStream,
   CSingleLock lock(*this);
 
   //Update any forced overlays.
-  for(VecOverlays::iterator it = m_overlays.begin(); it != m_overlays.end(); it++ )
+  for(VecOverlays::iterator it = m_overlays.begin(); it != m_overlays.end(); ++it )
   {
     if ((*it)->IsOverlayType(DVDOVERLAY_TYPE_SPU))
     {
index e075cd6..543d758 100644 (file)
@@ -63,7 +63,7 @@ double CPTSInputQueue::Get(int64_t bytes, bool consume)
   CSingleLock lock(m_sync);
 
   IT it = m_list.begin();
-  for(; it != m_list.end(); it++)
+  for(; it != m_list.end(); ++it)
   {
     if(bytes <= it->first)
     {
index 540d890..6aeacff 100644 (file)
@@ -246,7 +246,7 @@ void CDVDPlayerSubtitle::GetCurrentSubtitle(CStdString& strSubtitle, double pts)
   VecOverlays* pOverlays = m_pOverlayContainer->GetOverlays();
   if (pOverlays)
   {
-    for(vector<CDVDOverlay*>::iterator it = pOverlays->begin();it != pOverlays->end();it++)
+    for(vector<CDVDOverlay*>::iterator it = pOverlays->begin();it != pOverlays->end();++it)
     {
       CDVDOverlay* pOverlay = *it;
 
index 6ae8143..47e0fb6 100644 (file)
@@ -737,7 +737,7 @@ bool CEdl::AddCut(Cut& cut)
   else
   {
     vector<Cut>::iterator pCurrentCut;
-    for (pCurrentCut = m_vecCuts.begin(); pCurrentCut != m_vecCuts.end(); pCurrentCut++)
+    for (pCurrentCut = m_vecCuts.begin(); pCurrentCut != m_vecCuts.end(); ++pCurrentCut)
     {
       if (cut.start < pCurrentCut->start)
       {
index 87991f1..e4d9825 100644 (file)
@@ -46,9 +46,9 @@ CPlayerCoreFactory::CPlayerCoreFactory()
 
 CPlayerCoreFactory::~CPlayerCoreFactory()
 {
-  for(std::vector<CPlayerCoreConfig *>::iterator it = m_vecCoreConfigs.begin(); it != m_vecCoreConfigs.end(); it++)
+  for(std::vector<CPlayerCoreConfig *>::iterator it = m_vecCoreConfigs.begin(); it != m_vecCoreConfigs.end(); ++it)
     delete *it;
-  for(std::vector<CPlayerSelectionRule *>::iterator it = m_vecCoreSelectionRules.begin(); it != m_vecCoreSelectionRules.end(); it++)
+  for(std::vector<CPlayerSelectionRule *>::iterator it = m_vecCoreSelectionRules.begin(); it != m_vecCoreSelectionRules.end(); ++it)
     delete *it;
 }
 
@@ -321,7 +321,7 @@ bool CPlayerCoreFactory::LoadConfiguration(const std::string &file, bool clear)
 
   if (clear)
   {
-    for(std::vector<CPlayerCoreConfig *>::iterator it = m_vecCoreConfigs.begin(); it != m_vecCoreConfigs.end(); it++)
+    for(std::vector<CPlayerCoreConfig *>::iterator it = m_vecCoreConfigs.begin(); it != m_vecCoreConfigs.end(); ++it)
       delete *it;
     m_vecCoreConfigs.clear();
     // Builtin players; hard-coded because re-ordering them would break scripts
@@ -351,7 +351,7 @@ bool CPlayerCoreFactory::LoadConfiguration(const std::string &file, bool clear)
     m_vecCoreConfigs.push_back(omxplayer);
 #endif
 
-    for(std::vector<CPlayerSelectionRule *>::iterator it = m_vecCoreSelectionRules.begin(); it != m_vecCoreSelectionRules.end(); it++)
+    for(std::vector<CPlayerSelectionRule *>::iterator it = m_vecCoreSelectionRules.begin(); it != m_vecCoreSelectionRules.end(); ++it)
       delete *it;
     m_vecCoreSelectionRules.clear();
   }
@@ -427,7 +427,7 @@ void CPlayerCoreFactory::OnPlayerDiscovered(const CStdString& id, const CStdStri
   std::vector<CPlayerCoreConfig *>::iterator it;
   for(it  = m_vecCoreConfigs.begin();
       it != m_vecCoreConfigs.end();
-      it++)
+      ++it)
   {
     if ((*it)->GetId() == id)
     {
@@ -449,7 +449,7 @@ void CPlayerCoreFactory::OnPlayerRemoved(const CStdString& id)
   std::vector<CPlayerCoreConfig *>::iterator it;
   for(it  = m_vecCoreConfigs.begin();
       it != m_vecCoreConfigs.end();
-      it++)
+      ++it)
   {
     if ((*it)->GetId() == id)
       (*it)->m_eCore = EPC_NONE;