StreamProperties: getDuration returns 0 if the info is not available

Clement Champetier · Clement Champetier · commit 5e5e52dd41fe · 2016-06-20T16:14:35.000+02:00
diff --git a/src/AvTranscoder/properties/StreamProperties.cpp b/src/AvTranscoder/properties/StreamProperties.cpp
@@ -66,7 +66,10 @@ Rational StreamProperties::getTimeBase() const
 float StreamProperties::getDuration() const
 {
     const Rational timeBase = getTimeBase();
-    return av_q2d(timeBase) * _formatContext->streams[_streamIndex]->duration;
+    const size_t duration = _formatContext->streams[_streamIndex]->duration;
+    if(duration == (size_t)AV_NOPTS_VALUE)
+        return 0;
+    return av_q2d(timeBase) * duration;
 }
 
 AVMediaType StreamProperties::getStreamType() const
diff --git a/src/AvTranscoder/properties/StreamProperties.hpp b/src/AvTranscoder/properties/StreamProperties.hpp
@@ -19,7 +19,7 @@ class AvExport StreamProperties
     size_t getStreamIndex() const { return _streamIndex; }
     size_t getStreamId() const;
     Rational getTimeBase() const;
-    float getDuration() const; ///< in seconds
+    float getDuration() const; ///< in seconds, 0 if not available
     AVMediaType getStreamType() const;
 
     size_t getCodecId() const;

Original file line number	Diff line number	Diff line change
`@@ -66,7 +66,10 @@ Rational StreamProperties::getTimeBase() const`
`66`	`66`	`float StreamProperties::getDuration() const`
`67`	`67`	`{`
`68`	`68`	`const Rational timeBase = getTimeBase();`
`69`		`- return av_q2d(timeBase) * _formatContext->streams[_streamIndex]->duration;`
	`69`	`+ const size_t duration = _formatContext->streams[_streamIndex]->duration;`
	`70`	`+ if(duration == (size_t)AV_NOPTS_VALUE)`
	`71`	`+ return 0;`
	`72`	`+ return av_q2d(timeBase) * duration;`
`70`	`73`	`}`
`71`	`74`
`72`	`75`	`AVMediaType StreamProperties::getStreamType() const`