hotfix: store first AVCodecContext::timecode_frame_start

valnoel · valnoel · commit 81e811fd252f · 2015-05-12T18:40:02.000+02:00
* fixing VideoProperties::getStartTimecode method return value
diff --git a/src/AvTranscoder/mediaProperty/VideoProperties.cpp b/src/AvTranscoder/mediaProperty/VideoProperties.cpp
@@ -22,6 +22,7 @@ VideoProperties::VideoProperties( const FormatContext& formatContext, const size
 	, _pixelProperties()
 	, _isInterlaced( false )
 	, _isTopFieldFirst( false )
+	, _firstGopTimeCode( -1 )
 	, _gopStructure()
 {
 	if( _formatContext )
@@ -39,7 +40,10 @@ VideoProperties::VideoProperties( const FormatContext& formatContext, const size
 		_codec = avcodec_find_decoder( _codecContext->codec_id );
 
 	if( _codecContext )
+	{
 		_pixelProperties = PixelProperties( _codecContext->pix_fmt );
+		_firstGopTimeCode = _codecContext->timecode_frame_start;
+	}
 
 	if( level == eAnalyseLevelFirstGop )
 		analyseGopStructure( progress );
@@ -310,7 +314,7 @@ int64_t VideoProperties::getStartTimecode() const
 {
 	if( ! _codecContext )
 		throw std::runtime_error( "unknown codec context" );
-	return _codecContext->timecode_frame_start;
+	return _firstGopTimeCode;
 }
 
 std::string VideoProperties::getStartTimecodeString() const
diff --git a/src/AvTranscoder/mediaProperty/VideoProperties.hpp b/src/AvTranscoder/mediaProperty/VideoProperties.hpp
@@ -110,6 +110,12 @@ class AvExport VideoProperties : public StreamProperties
 	bool _isTopFieldFirst;
 	std::vector< std::pair< char, bool > > _gopStructure;
 	//@}
+
+	/**
+	 * @brief GOP timecode of the first frame
+	 * @note  AVCodecContext stores the GOP timecode of the last decoded frame
+	 */
+	int64_t _firstGopTimeCode;
 };
 
 }
