AudioProperties: get AVCodec attribute

Clement Champetier · Clement Champetier · commit 8df49bd40da8 · 2014-11-17T14:46:09.000+01:00
This removes multi call to avcodec_find_decoder().
diff --git a/src/AvTranscoder/mediaProperty/AudioProperties.cpp b/src/AvTranscoder/mediaProperty/AudioProperties.cpp
@@ -14,23 +14,25 @@ namespace avtranscoder
 AudioProperties::AudioProperties( const AVFormatContext* formatContext, const size_t index )
 	: _formatContext( formatContext )
 	, _codecContext( formatContext->streams[index]->codec )
+	, _codec( NULL )
 	, _streamId( index )
-{}
+{
+	if( _formatContext && _codecContext )
+		_codec = avcodec_find_decoder( _codecContext->codec_id );
+}
 
 std::string AudioProperties::getCodecName() const
 {
-	AVCodec* codec = avcodec_find_decoder( _codecContext->codec_id );
-	if( codec != NULL )
-		return std::string( codec->name );
+	if( _codec && _codec->name )
+		return std::string( _codec->name );
 	else
 		return "unknown codec";
 }
 
 std::string AudioProperties::getCodecLongName() const
 {
-	AVCodec* codec = avcodec_find_decoder( _codecContext->codec_id );
-	if( codec != NULL )
-		return std::string( codec->long_name );
+	if( _codec && _codec->long_name )
+		return std::string( _codec->long_name );
 	return "unknown codec";
 }
 
diff --git a/src/AvTranscoder/mediaProperty/AudioProperties.hpp b/src/AvTranscoder/mediaProperty/AudioProperties.hpp
@@ -43,6 +43,7 @@ class AvExport AudioProperties
 private:
 	const AVFormatContext* _formatContext;  ///< Has link (no ownership)
 	AVCodecContext* _codecContext;  ///< Has link (no ownership)
+	AVCodec* _codec; ///< Has link (no ownership)
 
 	size_t _streamId;
 	MetadatasMap _metadatas;
