encoders/decoders: do not log empty profile when setup

Clement Champetier · Clement Champetier · commit 8e5671e00c87 · 2015-11-04T16:01:40.000+01:00
diff --git a/src/AvTranscoder/decoder/AudioDecoder.cpp b/src/AvTranscoder/decoder/AudioDecoder.cpp
@@ -62,7 +62,10 @@ void AudioDecoder::setupDecoder( const ProfileLoader::Profile& profile )
 		throw std::runtime_error( msg );
 	}
 
-	LOG_INFO( "Setup audio decoder with:\n" << profile )
+	if( ! profile.empty() )
+	{
+		LOG_INFO( "Setup audio decoder with:\n" << profile )
+	}
 
 	AudioCodec& codec = _inputStream->getAudioCodec();
 
diff --git a/src/AvTranscoder/decoder/VideoDecoder.cpp b/src/AvTranscoder/decoder/VideoDecoder.cpp
@@ -60,7 +60,10 @@ void VideoDecoder::setupDecoder( const ProfileLoader::Profile& profile )
 		throw std::runtime_error( msg );
 	}
 
-	LOG_INFO( "Setup video decoder with:\n" << profile )
+	if( ! profile.empty() )
+	{
+		LOG_INFO( "Setup video decoder with:\n" << profile )
+	}
 
 	VideoCodec& codec = _inputStream->getVideoCodec();
 
diff --git a/src/AvTranscoder/encoder/AudioEncoder.cpp b/src/AvTranscoder/encoder/AudioEncoder.cpp
@@ -37,7 +37,10 @@ AudioEncoder::~AudioEncoder()
 
 void AudioEncoder::setupAudioEncoder( const AudioFrameDesc& frameDesc, const ProfileLoader::Profile& profile )
 {
-	LOG_INFO( "Setup audio encoder with:\n" << profile )
+	if( ! profile.empty() )
+	{
+		LOG_INFO( "Setup audio encoder with:\n" << profile )
+	}
 
 	// set sampleRate, number of channels, sample format
 	_codec.setAudioParameters( frameDesc );
diff --git a/src/AvTranscoder/encoder/VideoEncoder.cpp b/src/AvTranscoder/encoder/VideoEncoder.cpp
@@ -38,7 +38,10 @@ VideoEncoder::~VideoEncoder()
 
 void VideoEncoder::setupVideoEncoder( const VideoFrameDesc& frameDesc, const ProfileLoader::Profile& profile )
 {
-	LOG_INFO( "Setup video encoder with:\n" << profile )
+	if( ! profile.empty() )
+	{
+		LOG_INFO( "Setup video encoder with:\n" << profile )
+	}
 
 	// set width, height, pixel format, fps
 	_codec.setImageParameters( frameDesc );

Original file line number	Diff line number	Diff line change
`@@ -62,7 +62,10 @@ void AudioDecoder::setupDecoder( const ProfileLoader::Profile& profile )`
`62`	`62`	`throw std::runtime_error( msg );`
`63`	`63`	`}`
`64`	`64`
`65`		`- LOG_INFO( "Setup audio decoder with:\n" << profile )`
	`65`	`+ if( ! profile.empty() )`
	`66`	`+ {`
	`67`	`+ LOG_INFO( "Setup audio decoder with:\n" << profile )`
	`68`	`+ }`
`66`	`69`
`67`	`70`	`AudioCodec& codec = _inputStream->getAudioCodec();`
`68`	`71`
Original file line number	Diff line number	Diff line change
`@@ -60,7 +60,10 @@ void VideoDecoder::setupDecoder( const ProfileLoader::Profile& profile )`
`60`	`60`	`throw std::runtime_error( msg );`
`61`	`61`	`}`
`62`	`62`
`63`		`- LOG_INFO( "Setup video decoder with:\n" << profile )`
	`63`	`+ if( ! profile.empty() )`
	`64`	`+ {`
	`65`	`+ LOG_INFO( "Setup video decoder with:\n" << profile )`
	`66`	`+ }`
`64`	`67`
`65`	`68`	`VideoCodec& codec = _inputStream->getVideoCodec();`
`66`	`69`