VideoProperties: refactor how to launch deeper analysis

Clement Champetier · Clement Champetier · commit ad44be7ec50a · 2016-11-22T17:35:21.000+01:00
Use a switch statement instead of several if.
diff --git a/src/AvTranscoder/properties/VideoProperties.cpp b/src/AvTranscoder/properties/VideoProperties.cpp
@@ -38,13 +38,20 @@ VideoProperties::VideoProperties(const FileProperties& fileProperties, const siz
         _firstGopTimeCode = _codecContext->timecode_frame_start;
     }
 
-    if(_levelAnalysis > eAnalyseLevelHeader)
+    switch(_levelAnalysis)
     {
-        if(_levelAnalysis == eAnalyseLevelFirstGop)
+        case eAnalyseLevelFirstGop:
             analyseGopStructure(progress);
-        else if(_levelAnalysis == eAnalyseLevelFull)
+            break;
+        case eAnalyseLevelFull:
             analyseFull(progress);
+            break;
+        default:
+            break;
+    }
 
+    if(_levelAnalysis > eAnalyseLevelHeader)
+    {
         // Returns at the beginning of the stream
         const_cast<InputFile&>(_fileProperties->getInputFile()).seekAtFrame(0, AVSEEK_FLAG_BYTE);
     }
