VideoProperties: refactor how to getDuration

Clement Champetier · Clement Champetier · commit a61d95fb35b6 · 2016-11-22T18:26:19.000+01:00
Else statement was not necessary!
diff --git a/src/AvTranscoder/properties/VideoProperties.cpp b/src/AvTranscoder/properties/VideoProperties.cpp
@@ -475,18 +475,16 @@ float VideoProperties::getDuration() const
         LOG_INFO("Need a deeper analysis: see eAnalyseLevelFirstGop.")
         return 0;
     }
-    else
+
+    if(! _nbFrames)
     {
-        if(! _nbFrames)
-        {
-            LOG_INFO("Estimate the duration from the file size and the bitrate.")
-            const size_t bitRate = getBitRate();
-            if(bitRate)
-                return _fileProperties->getFileSize() / bitRate * 8;
-        }
-        LOG_INFO("Get the exact duration from the number of frames and the fps.")
-        return _nbFrames / getFps();
+        LOG_INFO("Estimate the duration from the file size and the bitrate.")
+        const size_t bitRate = getBitRate();
+        if(bitRate)
+            return _fileProperties->getFileSize() / bitRate * 8;
     }
+    LOG_INFO("Get the exact duration from the number of frames and the fps.")
+    return _nbFrames / getFps();
 }
 
 bool VideoProperties::hasBFrames() const
