[cosmetics] always create the save file state job on the heap
authorSam Stenvall <neggelandia@gmail.com>
Sat, 24 May 2014 12:14:44 +0000 (15:14 +0300)
committerSam Stenvall <neggelandia@gmail.com>
Sun, 13 Jul 2014 08:31:12 +0000 (11:31 +0300)
xbmc/Application.cpp

index 572707e..0a0d8c8 100644 (file)
@@ -4378,26 +4378,20 @@ void CApplication::SaveFileState(bool bForeground /* = false */)
   if (m_progressTrackingItem->IsPVRChannel() || !CProfilesManager::Get().GetCurrentProfile().canWriteDatabases())
     return;
 
+  CJob* job = new CSaveFileStateJob(*m_progressTrackingItem,
+      *m_stackFileItemToUpdate,
+      m_progressTrackingVideoResumeBookmark,
+      m_progressTrackingPlayCountUpdate,
+      CMediaSettings::Get().GetCurrentVideoSettings());
+  
   if (bForeground)
   {
-    CSaveFileStateJob job(*m_progressTrackingItem,
-    *m_stackFileItemToUpdate,
-    m_progressTrackingVideoResumeBookmark,
-    m_progressTrackingPlayCountUpdate,
-    CMediaSettings::Get().GetCurrentVideoSettings());
-
     // Run job in the foreground to make sure it finishes
-    job.DoWork();
+    job->DoWork();
+    delete job;
   }
   else
-  {
-    CJob* job = new CSaveFileStateJob(*m_progressTrackingItem,
-        *m_stackFileItemToUpdate,
-        m_progressTrackingVideoResumeBookmark,
-        m_progressTrackingPlayCountUpdate,
-        CMediaSettings::Get().GetCurrentVideoSettings());
     CJobManager::GetInstance().AddJob(job, NULL, CJob::PRIORITY_NORMAL);
-  }
 }
 
 void CApplication::UpdateFileState()