Fix subtitles (seen in DVB) which have a specified `to' time but
[dcpomatic.git] / src / lib / ffmpeg_examiner.cc
index 64866047863afc9de6b9f9893609d84958e4c723..271180d5d0970521e58e45f45fa087131abca9d3 100644 (file)
@@ -1,19 +1,20 @@
 /*
     Copyright (C) 2013-2015 Carl Hetherington <cth@carlh.net>
 
-    This program is free software; you can redistribute it and/or modify
+    This file is part of DCP-o-matic.
+
+    DCP-o-matic is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
     the Free Software Foundation; either version 2 of the License, or
     (at your option) any later version.
 
-    This program is distributed in the hope that it will be useful,
+    DCP-o-matic is distributed in the hope that it will be useful,
     but WITHOUT ANY WARRANTY; without even the implied warranty of
     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
     GNU General Public License for more details.
 
     You should have received a copy of the GNU General Public License
-    along with this program; if not, write to the Free Software
-    Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+    along with DCP-o-matic.  If not, see <http://www.gnu.org/licenses/>.
 
 */
 
@@ -29,7 +30,6 @@ extern "C" {
 #include "ffmpeg_audio_stream.h"
 #include "ffmpeg_subtitle_stream.h"
 #include "util.h"
-#include "safe_stringstream.h"
 #include <boost/foreach.hpp>
 #include <iostream>
 
@@ -61,9 +61,20 @@ FFmpegExaminer::FFmpegExaminer (shared_ptr<const FFmpegContent> c, shared_ptr<Jo
                                s->codec->channel_layout = av_get_default_channel_layout (s->codec->channels);
                        }
 
+                       DCPOMATIC_ASSERT (_format_context->duration != AV_NOPTS_VALUE);
+                       DCPOMATIC_ASSERT (s->codec->codec);
+                       DCPOMATIC_ASSERT (s->codec->codec->name);
+
                        _audio_streams.push_back (
                                shared_ptr<FFmpegAudioStream> (
-                                       new FFmpegAudioStream (audio_stream_name (s), s->id, s->codec->sample_rate, s->codec->channels)
+                                       new FFmpegAudioStream (
+                                               stream_name (s),
+                                               s->codec->codec->name,
+                                               s->id,
+                                               s->codec->sample_rate,
+                                               (double (_format_context->duration) / AV_TIME_BASE) * s->codec->sample_rate,
+                                               s->codec->channels
+                                               )
                                        )
                                );
 
@@ -72,17 +83,21 @@ FFmpegExaminer::FFmpegExaminer (shared_ptr<const FFmpegContent> c, shared_ptr<Jo
                }
        }
 
-       /* See if the header has duration information in it */
-       _need_video_length = _format_context->duration == AV_NOPTS_VALUE;
-       if (!_need_video_length) {
-               _video_length = (double (_format_context->duration) / AV_TIME_BASE) * video_frame_rate().get ();
+       if (has_video ()) {
+               /* See if the header has duration information in it */
+               _need_video_length = _format_context->duration == AV_NOPTS_VALUE;
+               if (!_need_video_length) {
+                       _video_length = (double (_format_context->duration) / AV_TIME_BASE) * video_frame_rate().get ();
+               }
        }
 
        if (job) {
                if (_need_video_length) {
                        job->sub (_("Finding length and subtitles"));
-               } else {
+               } else if (!_subtitle_streams.empty()) {
                        job->sub (_("Finding subtitles"));
+               } else {
+                       job->sub (_("Finding length"));
                }
        }
 
@@ -113,7 +128,7 @@ FFmpegExaminer::FFmpegExaminer (shared_ptr<const FFmpegContent> c, shared_ptr<Jo
 
                AVCodecContext* context = _format_context->streams[_packet.stream_index]->codec;
 
-               if (_packet.stream_index == _video_stream) {
+               if (_video_stream && _packet.stream_index == _video_stream.get()) {
                        video_packet (context);
                }
 
@@ -169,7 +184,7 @@ FFmpegExaminer::FFmpegExaminer (shared_ptr<const FFmpegContent> c, shared_ptr<Jo
           this is because we might not know the PTS offset when the first subtitle is seen.
           Now we know the PTS offset so we can apply it to those subtitles.
        */
-       if (video_frame_rate()) {
+       if (has_video() && video_frame_rate()) {
                BOOST_FOREACH (shared_ptr<FFmpegSubtitleStream> i, _subtitle_streams) {
                        i->add_offset (pts_offset (_audio_streams, _first_video, video_frame_rate().get()));
                }
@@ -179,6 +194,8 @@ FFmpegExaminer::FFmpegExaminer (shared_ptr<const FFmpegContent> c, shared_ptr<Jo
 void
 FFmpegExaminer::video_packet (AVCodecContext* context)
 {
+       DCPOMATIC_ASSERT (_video_stream);
+
        if (_first_video && !_need_video_length) {
                return;
        }
@@ -186,11 +203,11 @@ FFmpegExaminer::video_packet (AVCodecContext* context)
        int frame_finished;
        if (avcodec_decode_video2 (context, _frame, &frame_finished, &_packet) >= 0 && frame_finished) {
                if (!_first_video) {
-                       _first_video = frame_time (_format_context->streams[_video_stream]);
+                       _first_video = frame_time (_format_context->streams[_video_stream.get()]);
                }
                if (_need_video_length) {
                        _video_length = frame_time (
-                               _format_context->streams[_video_stream]
+                               _format_context->streams[_video_stream.get()]
                                ).get_value_or (ContentTime ()).frames_round (video_frame_rate().get ());
                }
        }
@@ -219,36 +236,41 @@ FFmpegExaminer::subtitle_packet (AVCodecContext* context, shared_ptr<FFmpegSubti
                FFmpegSubtitlePeriod const period = subtitle_period (sub);
                bool const starts_image = subtitle_starts_image (sub);
 
+               /* Some streams (notably DVB streams) have subtitles which have a specified end time
+                  but which are then stopped earlier than this by a zero-num_rect subtitle.
+               */
+
                LastSubtitleMap::iterator last = _last_subtitle_start.find (stream);
-               if (last != _last_subtitle_start.end() && last->second) {
-                       /* We have seen the start of a subtitle but not yet the end.  Whatever this is
-                          finishes the previous subtitle, so add it */
-                       if (last->second->image) {
-                               stream->add_image_subtitle (last->second->id, ContentTimePeriod (last->second->time, period.from));
-                       } else {
-                               stream->add_text_subtitle (last->second->id, ContentTimePeriod (last->second->time, period.from));
-                       }
-                       if (sub.num_rects == 0) {
-                               /* This is a `proper' end-of-subtitle */
-                               _last_subtitle_start[stream] = optional<SubtitleStart> ();
+               if (sub.num_rects == 0 && last != _last_subtitle_start.end() && last->second) {
+                       /* Set (or fix) the `to' time for the last subtitle */
+                       stream->set_subtitle_to (last->second->id, period.from);
+                       _last_subtitle_start[stream] = optional<SubtitleStart> ();
+               } else if (sub.num_rects > 0) {
+                       /* Add a subtitle; if we don't know the `to' time we set it to the from time and fix it later */
+                       if (starts_image) {
+                               stream->add_image_subtitle (id, ContentTimePeriod (period.from, period.to.get_value_or (period.from)));
                        } else {
-                               /* This is just another subtitle, so we start again */
-                               _last_subtitle_start[stream] = SubtitleStart (id, starts_image, period.from);
-                       }
-               } else if (sub.num_rects == 1) {
-                       if (period.to) {
-                               if (starts_image) {
-                                       stream->add_image_subtitle (id, ContentTimePeriod (period.from, period.to.get ()));
-                               } else {
-                                       stream->add_text_subtitle (id, ContentTimePeriod (period.from, period.to.get ()));
-                               }
-                       } else {
-                               _last_subtitle_start[stream] = SubtitleStart (id, starts_image, period.from);
+                               stream->add_text_subtitle (id, ContentTimePeriod (period.from, period.to.get_value_or (period.from)));
                        }
+
+                       _last_subtitle_start[stream] = SubtitleStart (id, starts_image, period.from);
                }
 
                for (unsigned int i = 0; i < sub.num_rects; ++i) {
                        if (sub.rects[i]->type == SUBTITLE_BITMAP) {
+#ifdef DCPOMATIC_HAVE_AVSUBTITLERECT_PICT
+                               uint32_t* palette = (uint32_t *) sub.rects[i]->pict.data[1];
+                               for (int j = 0; j < sub.rects[i]->nb_colors; ++j) {
+                                       RGBA rgba  (
+                                               (palette[j] & 0x00ff0000) >> 16,
+                                               (palette[j] & 0x0000ff00) >> 8,
+                                               (palette[j] & 0x000000ff) >> 0,
+                                               (palette[j] & 0xff000000) >> 24
+                                               );
+
+                                       stream->set_colour (rgba, rgba);
+                               }
+#else
                                uint32_t* palette = (uint32_t *) sub.rects[i]->data[1];
                                for (int j = 0; j < sub.rects[i]->nb_colors; ++j) {
                                        RGBA rgba  (
@@ -260,6 +282,7 @@ FFmpegExaminer::subtitle_packet (AVCodecContext* context, shared_ptr<FFmpegSubti
 
                                        stream->set_colour (rgba, rgba);
                                }
+#endif
                        }
                }
 
@@ -283,11 +306,12 @@ FFmpegExaminer::frame_time (AVStream* s) const
 optional<double>
 FFmpegExaminer::video_frame_rate () const
 {
+       DCPOMATIC_ASSERT (_video_stream);
        /* This use of r_frame_rate is debateable; there's a few different
         * frame rates in the format context, but this one seems to be the most
         * reliable.
         */
-       return av_q2d (av_stream_get_r_frame_rate (_format_context->streams[_video_stream]));
+       return av_q2d (av_stream_get_r_frame_rate (_format_context->streams[_video_stream.get()]));
 }
 
 dcp::Size
@@ -306,7 +330,8 @@ FFmpegExaminer::video_length () const
 optional<double>
 FFmpegExaminer::sample_aspect_ratio () const
 {
-       AVRational sar = av_guess_sample_aspect_ratio (_format_context, _format_context->streams[_video_stream], 0);
+       DCPOMATIC_ASSERT (_video_stream);
+       AVRational sar = av_guess_sample_aspect_ratio (_format_context, _format_context->streams[_video_stream.get()], 0);
        if (sar.num == 0) {
                /* I assume this means that we don't know */
                return optional<double> ();
@@ -314,57 +339,39 @@ FFmpegExaminer::sample_aspect_ratio () const
        return double (sar.num) / sar.den;
 }
 
-string
-FFmpegExaminer::audio_stream_name (AVStream* s) const
-{
-       SafeStringStream n;
-
-       n << stream_name (s);
-
-       if (!n.str().empty()) {
-               n << "; ";
-       }
-
-       n << s->codec->channels << " channels";
-
-       return n.str ();
-}
-
 string
 FFmpegExaminer::subtitle_stream_name (AVStream* s) const
 {
-       SafeStringStream n;
-
-       n << stream_name (s);
+       string n = stream_name (s);
 
-       if (n.str().empty()) {
-               n << _("unknown");
+       if (n.empty()) {
+               n = _("unknown");
        }
 
-       return n.str ();
+       return n;
 }
 
 string
 FFmpegExaminer::stream_name (AVStream* s) const
 {
-       SafeStringStream n;
+       string n;
 
        if (s->metadata) {
                AVDictionaryEntry const * lang = av_dict_get (s->metadata, "language", 0, 0);
                if (lang) {
-                       n << lang->value;
+                       n = lang->value;
                }
 
                AVDictionaryEntry const * title = av_dict_get (s->metadata, "title", 0, 0);
                if (title) {
-                       if (!n.str().empty()) {
-                               n << " ";
+                       if (!n.empty()) {
+                               n += " ";
                        }
-                       n << title->value;
+                       n += title->value;
                }
        }
 
-       return n.str ();
+       return n;
 }
 
 int
@@ -459,3 +466,9 @@ FFmpegExaminer::yuv () const
                return false;
        }
 }
+
+bool
+FFmpegExaminer::has_video () const
+{
+       return static_cast<bool> (_video_stream);
+}