Transcoder: manage FFmpeg log level

Clement Champetier · Clement Champetier · commit 4be8bceec20f · 2014-12-03T17:05:16.000+01:00
* Set it to AV_LOG_QUIET in Transcoder constructor.
* Set it to AV_LOG_DEBUG if setVerbose of Transcoder to true.
diff --git a/src/AvTranscoder/transcoder/Transcoder.cpp b/src/AvTranscoder/transcoder/Transcoder.cpp
@@ -22,7 +22,11 @@ Transcoder::Transcoder( OutputFile& outputFile )
 	, _mainStreamIndex( 0 )
 	, _verbose( false )
 {
+	// Initialize the OutputFile
 	_outputFile.setup();
+
+	// Print no output from ffmpeg
+	av_log_set_level( AV_LOG_QUIET );
 }
 
 Transcoder::~Transcoder()
@@ -285,9 +289,6 @@ void Transcoder::process( IProgress& progress )
 	
 	_outputFile.beginWrap();
 
-	if( _verbose )
-		av_log_set_level( AV_LOG_DEBUG );
-
 	double totalDuration = getTotalDurationFromProcessMethod();
 
 	size_t frame = 0;
@@ -325,6 +326,10 @@ void Transcoder::setVerbose( bool verbose )
 		(*it)->setVerbose( _verbose );
 	}
 	_outputFile.setVerbose( _verbose );
+
+	// Print stuff which is only useful for ffmpeg developers.
+	if( _verbose )
+		av_log_set_level( AV_LOG_DEBUG );
 }
 
 void Transcoder::addRewrapStream( const std::string& filename, const size_t streamIndex )
diff --git a/src/AvTranscoder/transcoder/Transcoder.hpp b/src/AvTranscoder/transcoder/Transcoder.hpp
@@ -43,6 +43,9 @@ enum EProcessMethod
 class AvExport Transcoder
 {
 public:
+	/**
+	 * @note Set FFmpeg log level to quite.
+	 */
 	Transcoder( OutputFile& outputFile );
 	
 	~Transcoder();
@@ -131,8 +134,9 @@ class AvExport Transcoder
 	void setProcessMethod( const EProcessMethod eProcessMethod, const size_t indexBasedStream = 0 );
 
 	/**
-	 * @brief Set verbose mode for the Transcoder and its streams.
+	 * @brief Set verbose mode for the Transcoder, its streams, and its output file.
 	 * @note If you call it before adding the streams, no verbose mode will be set for the new streams.
+	 * @note set av log level to AV_LOG_DEBUG
 	 */
 	void setVerbose( bool verbose = true );
 
