X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Flib%2Fffmpeg_examiner.cc;h=0a236d836be46b4f5baacdee175201e154de6cab;hb=5f206d32ff60148ab72b35d5823f56bdbb7f50bf;hp=0f2b89f3d0f9e88c6f5819cba7cb204431877bef;hpb=81e36a3a409ddbcc79986fc20f37a8ef720771f9;p=dcpomatic.git diff --git a/src/lib/ffmpeg_examiner.cc b/src/lib/ffmpeg_examiner.cc index 0f2b89f3d..0a236d836 100644 --- a/src/lib/ffmpeg_examiner.cc +++ b/src/lib/ffmpeg_examiner.cc @@ -1,5 +1,5 @@ /* - Copyright (C) 2013-2015 Carl Hetherington + Copyright (C) 2013-2021 Carl Hetherington This file is part of DCP-o-matic. @@ -18,6 +18,7 @@ */ + #include "dcpomatic_log.h" #include "ffmpeg_examiner.h" #include "ffmpeg_content.h" @@ -32,19 +33,22 @@ extern "C" { #include #include #include -#include +#include #include +#include } DCPOMATIC_ENABLE_WARNINGS #include #include "i18n.h" -using std::string; + using std::cout; +using std::make_shared; using std::max; -using std::vector; using std::shared_ptr; +using std::string; +using std::vector; using boost::optional; using namespace dcpomatic; @@ -65,37 +69,34 @@ FFmpegExaminer::FFmpegExaminer (shared_ptr c, shared_ptrnb_streams; ++i) { - AVStream* s = _format_context->streams[i]; -DCPOMATIC_DISABLE_WARNINGS - if (s->codec->codec_type == AVMEDIA_TYPE_AUDIO) { + auto s = _format_context->streams[i]; + auto codec = _codec_context[i] ? _codec_context[i]->codec : nullptr; + if (s->codecpar->codec_type == AVMEDIA_TYPE_AUDIO && codec) { /* This is a hack; sometimes it seems that _audio_codec_context->channel_layout isn't set up, so bodge it here. No idea why we should have to do this. */ - if (s->codec->channel_layout == 0) { - s->codec->channel_layout = av_get_default_channel_layout (s->codec->channels); + if (s->codecpar->channel_layout == 0) { + s->codecpar->channel_layout = av_get_default_channel_layout (s->codecpar->channels); } DCPOMATIC_ASSERT (_format_context->duration != AV_NOPTS_VALUE); - DCPOMATIC_ASSERT (s->codec->codec); - DCPOMATIC_ASSERT (s->codec->codec->name); + DCPOMATIC_ASSERT (codec->name); _audio_streams.push_back ( - shared_ptr ( - new FFmpegAudioStream ( - stream_name (s), - s->codec->codec->name, - s->id, - s->codec->sample_rate, - llrint ((double (_format_context->duration) / AV_TIME_BASE) * s->codec->sample_rate), - s->codec->channels - ) + make_shared( + stream_name (s), + codec->name, + s->id, + s->codecpar->sample_rate, + llrint ((double(_format_context->duration) / AV_TIME_BASE) * s->codecpar->sample_rate), + s->codecpar->channels ) ); - } else if (s->codec->codec_type == AVMEDIA_TYPE_SUBTITLE) { - _subtitle_streams.push_back (shared_ptr (new FFmpegSubtitleStream (subtitle_stream_name (s), s->id))); + } else if (s->codecpar->codec_type == AVMEDIA_TYPE_SUBTITLE) { + _subtitle_streams.push_back (make_shared(subtitle_stream_name (s), s->id)); } } @@ -124,8 +125,11 @@ DCPOMATIC_DISABLE_WARNINGS */ string temporal_reference; while (true) { - int r = av_read_frame (_format_context, &_packet); + auto packet = av_packet_alloc (); + DCPOMATIC_ASSERT (packet); + int r = av_read_frame (_format_context, packet); if (r < 0) { + av_packet_free (&packet); break; } @@ -137,25 +141,24 @@ DCPOMATIC_DISABLE_WARNINGS } } - auto context = _format_context->streams[_packet.stream_index]->codec; -DCPOMATIC_ENABLE_WARNINGS + auto context = _codec_context[packet->stream_index]; - if (_video_stream && _packet.stream_index == _video_stream.get()) { - video_packet (context, temporal_reference); + if (_video_stream && packet->stream_index == _video_stream.get()) { + video_packet (context, temporal_reference, packet); } bool got_all_audio = true; for (size_t i = 0; i < _audio_streams.size(); ++i) { - if (_audio_streams[i]->uses_index (_format_context, _packet.stream_index)) { - audio_packet (context, _audio_streams[i]); + if (_audio_streams[i]->uses_index(_format_context, packet->stream_index)) { + audio_packet (context, _audio_streams[i], packet); } if (!_audio_streams[i]->first_audio) { got_all_audio = false; } } - av_packet_unref (&_packet); + av_packet_free (&packet); if (_first_video && got_all_audio && temporal_reference.size() >= (PULLDOWN_CHECK_FRAMES * 2)) { /* All done */ @@ -163,23 +166,20 @@ DCPOMATIC_ENABLE_WARNINGS } } - _packet.data = nullptr; - _packet.size = 0; - /* XXX: I'm not sure this makes any sense: how does _packet.stream_index get the right value here? */ -DCPOMATIC_DISABLE_WARNINGS - auto context = _format_context->streams[_packet.stream_index]->codec; -DCPOMATIC_ENABLE_WARNINGS - while (video_packet(context, temporal_reference)) {} - for (size_t i = 0; i < _audio_streams.size(); ++i) { - if (_audio_streams[i]->uses_index (_format_context, _packet.stream_index)) { - audio_packet (context, _audio_streams[i]); - } + if (_video_stream) { + auto context = _codec_context[_video_stream.get()]; + while (video_packet(context, temporal_reference, nullptr)) {} + } + + for (auto i: _audio_streams) { + auto context = _codec_context[i->index(_format_context)]; + audio_packet(context, i, nullptr); } if (_video_stream) { /* This code taken from get_rotation() in ffmpeg:cmdutils.c */ - AVStream* stream = _format_context->streams[*_video_stream]; - AVDictionaryEntry* rotate_tag = av_dict_get (stream->metadata, "rotate", 0, 0); + auto stream = _format_context->streams[*_video_stream]; + auto rotate_tag = av_dict_get (stream->metadata, "rotate", 0, 0); uint8_t* displaymatrix = av_stream_get_side_data (stream, AV_PKT_DATA_DISPLAYMATRIX, 0); _rotation = 0; @@ -213,7 +213,7 @@ DCPOMATIC_ENABLE_WARNINGS * @return true if some video was decoded, otherwise false. */ bool -FFmpegExaminer::video_packet (AVCodecContext* context, string& temporal_reference) +FFmpegExaminer::video_packet (AVCodecContext* context, string& temporal_reference, AVPacket* packet) { DCPOMATIC_ASSERT (_video_stream); @@ -221,12 +221,22 @@ FFmpegExaminer::video_packet (AVCodecContext* context, string& temporal_referenc return false; } - int frame_finished; -DCPOMATIC_DISABLE_WARNINGS - if (avcodec_decode_video2 (context, _frame, &frame_finished, &_packet) < 0 || !frame_finished) { + int r = avcodec_send_packet (context, packet); + if (r < 0 && !(r == AVERROR_EOF && !packet)) { + /* We could cope with AVERROR(EAGAIN) and re-send the packet but I think it should never happen. + * AVERROR_EOF can happen during flush if we've already sent a flush packet. + */ + throw DecodeError (N_("avcodec_send_packet"), N_("FFmpegExaminer::video_packet"), r); + } + + r = avcodec_receive_frame (context, _frame); + if (r == AVERROR(EAGAIN)) { + /* More input is required */ + return true; + } else if (r == AVERROR_EOF) { + /* No more output is coming */ return false; } -DCPOMATIC_ENABLE_WARNINGS if (!_first_video) { _first_video = frame_time (_format_context->streams[_video_stream.get()]); @@ -246,18 +256,26 @@ DCPOMATIC_ENABLE_WARNINGS void -FFmpegExaminer::audio_packet (AVCodecContext* context, shared_ptr stream) +FFmpegExaminer::audio_packet (AVCodecContext* context, shared_ptr stream, AVPacket* packet) { if (stream->first_audio) { return; } - int frame_finished; -DCPOMATIC_DISABLE_WARNINGS - if (avcodec_decode_audio4 (context, _frame, &frame_finished, &_packet) >= 0 && frame_finished) { -DCPOMATIC_ENABLE_WARNINGS - stream->first_audio = frame_time (stream->stream (_format_context)); + int r = avcodec_send_packet (context, packet); + if (r < 0 && !(r == AVERROR_EOF && !packet) && r != AVERROR(EAGAIN)) { + /* We could cope with AVERROR(EAGAIN) and re-send the packet but I think it should never happen. + * AVERROR_EOF can happen during flush if we've already sent a flush packet. + * EAGAIN means we need to do avcodec_receive_frame, so just carry on and do that. + */ + throw DecodeError (N_("avcodec_send_packet"), N_("FFmpegExaminer::audio_packet"), r); + } + + if (avcodec_receive_frame (context, _frame) < 0) { + return; } + + stream->first_audio = frame_time (stream->stream(_format_context)); } @@ -266,9 +284,7 @@ FFmpegExaminer::frame_time (AVStream* s) const { optional t; -DCPOMATIC_DISABLE_WARNINGS - int64_t const bet = av_frame_get_best_effort_timestamp (_frame); -DCPOMATIC_ENABLE_WARNINGS + int64_t const bet = _frame->best_effort_timestamp; if (bet != AV_NOPTS_VALUE) { t = ContentTime::from_seconds (bet * av_q2d (s->time_base)); } @@ -284,12 +300,14 @@ FFmpegExaminer::video_frame_rate () const return av_q2d(av_guess_frame_rate(_format_context, _format_context->streams[_video_stream.get()], 0)); } + dcp::Size FFmpegExaminer::video_size () const { return dcp::Size (video_codec_context()->width, video_codec_context()->height); } + /** @return Length according to our content's header */ Frame FFmpegExaminer::video_length () const @@ -297,22 +315,24 @@ FFmpegExaminer::video_length () const return max (Frame (1), _video_length); } + optional FFmpegExaminer::sample_aspect_ratio () const { DCPOMATIC_ASSERT (_video_stream); - AVRational sar = av_guess_sample_aspect_ratio (_format_context, _format_context->streams[_video_stream.get()], 0); + auto 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 (); + return {}; } return double (sar.num) / sar.den; } + string FFmpegExaminer::subtitle_stream_name (AVStream* s) const { - string n = stream_name (s); + auto n = stream_name (s); if (n.empty()) { n = _("unknown"); @@ -321,18 +341,19 @@ FFmpegExaminer::subtitle_stream_name (AVStream* s) const return n; } + string FFmpegExaminer::stream_name (AVStream* s) const { string n; if (s->metadata) { - AVDictionaryEntry const * lang = av_dict_get (s->metadata, "language", 0, 0); + auto const lang = av_dict_get (s->metadata, "language", 0, 0); if (lang) { n = lang->value; } - AVDictionaryEntry const * title = av_dict_get (s->metadata, "title", 0, 0); + auto const title = av_dict_get (s->metadata, "title", 0, 0); if (title) { if (!n.empty()) { n += " "; @@ -344,18 +365,20 @@ FFmpegExaminer::stream_name (AVStream* s) const return n; } + optional FFmpegExaminer::bits_per_pixel () const { if (video_codec_context()->pix_fmt == -1) { - return optional(); + return {}; } - AVPixFmtDescriptor const * d = av_pix_fmt_desc_get (video_codec_context()->pix_fmt); + auto const d = av_pix_fmt_desc_get (video_codec_context()->pix_fmt); DCPOMATIC_ASSERT (d); return av_get_bits_per_pixel (d); } + bool FFmpegExaminer::yuv () const { @@ -437,12 +460,14 @@ FFmpegExaminer::yuv () const } } + bool FFmpegExaminer::has_video () const { - return static_cast (_video_stream); + return static_cast(_video_stream); } + VideoRange FFmpegExaminer::range () const { @@ -455,3 +480,12 @@ FFmpegExaminer::range () const return VideoRange::FULL; } } + + +PixelQuanta +FFmpegExaminer::pixel_quanta () const +{ + auto const desc = av_pix_fmt_desc_get(video_codec_context()->pix_fmt); + return { 1 << desc->log2_chroma_w, 1 << desc->log2_chroma_h }; +} +