Transcoder: call processFrame before checking the end of process

Clement Champetier · Clement Champetier · commit d8f283cdc38d · 2016-02-29T17:43:00.000+01:00
* Because in some cases, the duration of input is 0 and we would like to process at least one frame. * Fix #231
diff --git a/src/AvTranscoder/transcoder/Transcoder.cpp b/src/AvTranscoder/transcoder/Transcoder.cpp
@@ -289,6 +289,10 @@ ProcessStat Transcoder::process(IProgress& progress)
     bool frameProcessed = true;
     while(frameProcessed)
     {
+        LOG_DEBUG("Process frame " << frame)
+        frameProcessed = processFrame();
+        ++frame;
+
         const float progressDuration = getCurrentOutputDuration();
 
         // check if JobStatusCancel
@@ -306,10 +310,6 @@ ProcessStat Transcoder::process(IProgress& progress)
                      << progressDuration << "s) is equal or upper than " << expectedOutputDuration << "s.")
             break;
         }
-
-        LOG_DEBUG("Process frame " << frame)
-        frameProcessed = processFrame();
-        ++frame;
     }
 
     _outputFile.endWrap();

Original file line number	Diff line number	Diff line change
`@@ -289,6 +289,10 @@ ProcessStat Transcoder::process(IProgress& progress)`
`289`	`289`	`bool frameProcessed = true;`
`290`	`290`	`while(frameProcessed)`
`291`	`291`	`{`
	`292`	`+ LOG_DEBUG("Process frame " << frame)`
	`293`	`+ frameProcessed = processFrame();`
	`294`	`+ ++frame;`
	`295`	`+`
`292`	`296`	`const float progressDuration = getCurrentOutputDuration();`
`293`	`297`
`294`	`298`	`// check if JobStatusCancel`
`@@ -306,10 +310,6 @@ ProcessStat Transcoder::process(IProgress& progress)`
`306`	`310`	`<< progressDuration << "s) is equal or upper than " << expectedOutputDuration << "s.")`
`307`	`311`	`break;`
`308`	`312`	`}`
`309`		`-`
`310`		`- LOG_DEBUG("Process frame " << frame)`
`311`		`- frameProcessed = processFrame();`
`312`		`- ++frame;`
`313`	`313`	`}`
`314`	`314`
`315`	`315`	`_outputFile.endWrap();`