Hand-apply cb998bee2e70f3426fd95655736deae803efa58e from master; examine a whole...

[dcpomatic.git] / src / lib / ffmpeg_examiner.cc

diff --git a/src/lib/ffmpeg_examiner.cc b/src/lib/ffmpeg_examiner.cc
index 46e93b16c322e9d469635b9d5e82cee11f366f3c..f28060a56da356d30f48295db8c4e9d0444cb56a 100644 (file)
--- a/src/lib/ffmpeg_examiner.cc
+++ b/src/lib/ffmpeg_examiner.cc
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2013-2014 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2013-2015 Carl Hetherington <cth@carlh.net>
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
@@ -38,6 +38,7 @@ using boost::optional;
 
 FFmpegExaminer::FFmpegExaminer (shared_ptr<const FFmpegContent> c)
        : FFmpeg (c)
+       , _need_video_length (false)
 {
        /* Find audio and subtitle streams */
 
@@ -64,6 +65,12 @@ FFmpegExaminer::FFmpegExaminer (shared_ptr<const FFmpegContent> c)
                }
        }
 
+       /* See if the header has duration information in it */
+       _need_video_length = _format_context->duration == AV_NOPTS_VALUE;
+       if (!_need_video_length) {
+               _video_length = ContentTime::from_seconds (double (_format_context->duration) / AV_TIME_BASE);
+       }
+
        /* Run through until we find:
         *   - the first video.
         *   - the first audio for each stream.
@@ -104,13 +111,18 @@ FFmpegExaminer::FFmpegExaminer (shared_ptr<const FFmpegContent> c)
 void
 FFmpegExaminer::video_packet (AVCodecContext* context)
 {
-       if (_first_video) {
+       if (_first_video && !_need_video_length) {
                return;
        }
 
        int frame_finished;
        if (avcodec_decode_video2 (context, _frame, &frame_finished, &_packet) >= 0 && frame_finished) {
-               _first_video = frame_time (_format_context->streams[_video_stream]);
+               if (!_first_video) {
+                       _first_video = frame_time (_format_context->streams[_video_stream]);
+               }
+               if (_need_video_length) {
+                       _video_length = frame_time (_format_context->streams[_video_stream]).get_value_or (ContentTime ());
+               }
        }
 }
 
@@ -156,7 +168,7 @@ FFmpegExaminer::frame_time (AVStream* s) const
        return t;
 }
 
-float
+optional<float>
 FFmpegExaminer::video_frame_rate () const
 {
        /* This use of r_frame_rate is debateable; there's a few different
@@ -177,13 +189,17 @@ ContentTime
 FFmpegExaminer::video_length () const
 {
        ContentTime const length = ContentTime::from_seconds (double (_format_context->duration) / AV_TIME_BASE);
-       return ContentTime (max (ContentTime::Type (1), length.get ()));
+       return ContentTime (max (ContentTime (1), _video_length));
 }
 
 optional<float>
 FFmpegExaminer::sample_aspect_ratio () const
 {
        AVRational sar = av_guess_sample_aspect_ratio (_format_context, _format_context->streams[_video_stream], 0);
+       if (sar.num == 0) {
+               /* I assume this means that we don't know */
+               return optional<float> ();
+       }
        return float (sar.num) / sar.den;
 }