From d2a7c8d3db669bafb5a86d2b61dd74ddcb553a2f Mon Sep 17 00:00:00 2001
From: hidefuku <fukkun1-11@hotmail.co.jp>
Date: Wed, 29 Mar 2017 00:19:50 +0900
Subject: [PATCH] Fixed bug a freeze when try export without ffmpeg.

---
 src/ctrl/Exporter.cpp | 22 ++++++++++++++++++----
 1 file changed, 18 insertions(+), 4 deletions(-)

diff --git a/src/ctrl/Exporter.cpp b/src/ctrl/Exporter.cpp
index 878576e7..800ba6bd 100644
--- a/src/ctrl/Exporter.cpp
+++ b/src/ctrl/Exporter.cpp
@@ -144,12 +144,12 @@ bool Exporter::FFMpeg::finish(const std::function<bool()>& aWaiter)
 
     while (!mProcess->waitForFinished(kMSec))
     {
-        if (!aWaiter() || mFinished) break;
+        if (!aWaiter() || mFinished || mErrorOccurred) break;
     }
 
     auto exitStatus = mProcess->exitStatus();
     //qDebug() << "exit status" << exitStatus;
-    //qDebug() << "exit code" << mFFMpeg->exitCode();
+    //qDebug() << "exit code" << mProcess->exitCode();
 
     mProcess.reset();
     return (exitStatus == QProcess::NormalExit);
@@ -452,6 +452,14 @@ Exporter::Result Exporter::execute()
         }
     }
 
+    if (mFFMpeg.errorOccurred())
+    {
+        mLog = "FFmpeg error occurred.\n" + mFFMpeg.errorString();
+        return mFFMpeg.errorCode() == QProcess::FailedToStart ?
+                    Result(ResultCode_FFMpegFailedToStart, mLog) :
+                    Result(ResultCode_FFMpegError, mLog);
+    }
+
     return finish();
 }
 
@@ -521,7 +529,10 @@ bool Exporter::updateTime(core::TimeInfo& aDst)
 
 bool Exporter::update()
 {
-    if (!mExporting) return false;
+    if (!mExporting)
+    {
+        return false;
+    }
 
     const int currentIndex = mIndex;
 
@@ -608,7 +619,10 @@ bool Exporter::update()
         updateLog();
 
         // export
-        exportImage(outImage, currentIndex);
+        if (!exportImage(outImage, currentIndex))
+        {
+            return false;
+        }
     }
 
     return true;