VideoProperties: add log when bitRate is not available in the container

Clement Champetier · Clement Champetier · commit 14120ec5e045 · 2016-11-03T14:04:30.000+01:00
diff --git a/src/AvTranscoder/properties/VideoProperties.cpp b/src/AvTranscoder/properties/VideoProperties.cpp
@@ -329,9 +329,10 @@ size_t VideoProperties::getBitRate() const
     if(_codecContext->bit_rate || _codecContext->rc_max_rate)
         return _codecContext->bit_rate;
 
+    LOG_WARN("The bitrate of the stream '" << _streamIndex << "' of file '" << _formatContext->filename << "' is unknown.")
     if(_levelAnalysis == eAnalyseLevelHeader)
     {
-        LOG_WARN("The bitrate of the stream '" << _streamIndex << "' of file '" << _formatContext->filename << "' is unknown.")
+        LOG_INFO("Need a deeper analysis: see eAnalyseLevelFirstGop.");
         return 0;
     }
 
diff --git a/src/AvTranscoder/properties/VideoProperties.hpp b/src/AvTranscoder/properties/VideoProperties.hpp
@@ -42,7 +42,9 @@ class AvExport VideoProperties : public StreamProperties
 
     /**
      * @return The video bitrate in bits/s.
-     * @warning If there is no such info available in the container, this data is estimated by decoding the first GOP.
+     * @note 0 if unknown.
+     * @warning If there is no such info available in the container, this data is estimated using the information of the first GOP.
+     * @see eAnalyseLevelFirstGop
      */
     size_t getBitRate() const;
     size_t getMaxBitRate() const;

Original file line number	Diff line number	Diff line change
`@@ -329,9 +329,10 @@ size_t VideoProperties::getBitRate() const`
`329`	`329`	`if(_codecContext->bit_rate \|\| _codecContext->rc_max_rate)`
`330`	`330`	`return _codecContext->bit_rate;`
`331`	`331`
	`332`	`+ LOG_WARN("The bitrate of the stream '" << _streamIndex << "' of file '" << _formatContext->filename << "' is unknown.")`
`332`	`333`	`if(_levelAnalysis == eAnalyseLevelHeader)`
`333`	`334`	`{`
`334`		`- LOG_WARN("The bitrate of the stream '" << _streamIndex << "' of file '" << _formatContext->filename << "' is unknown.")`
	`335`	`+ LOG_INFO("Need a deeper analysis: see eAnalyseLevelFirstGop.");`
`335`	`336`	`return 0;`
`336`	`337`	`}`
`337`	`338`