X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;f=nageru%2Fffmpeg_capture.cpp;h=4b9d477648cdbc8ee7e3efedfb79e112e972a2b3;hb=8bf76def09e0747734c33a21a8f94b3f462305c7;hp=9af4e833f76340bfa3b92717f78e3763fad02776;hpb=22d74c9f49f8602fd33c499a575a3fa605f43dee;p=nageru

diff --git a/nageru/ffmpeg_capture.cpp b/nageru/ffmpeg_capture.cpp
index 9af4e83..4b9d477 100644
--- a/nageru/ffmpeg_capture.cpp
+++ b/nageru/ffmpeg_capture.cpp
@@ -50,6 +50,11 @@ using namespace bmusb;
 using namespace movit;
 using namespace Eigen;
 
+// Avoid deprecation warnings, but we don't want to drop FFmpeg 5.1 support just yet.
+#if LIBAVUTIL_VERSION_INT >= AV_VERSION_INT(57, 30, 100)
+#define pkt_duration duration
+#endif
+
 namespace {
 
 steady_clock::time_point compute_frame_start(int64_t frame_pts, int64_t pts_origin, const AVRational &video_timebase, const steady_clock::time_point &origin, double rate)
@@ -901,34 +906,31 @@ AVFrameWithDeleter FFmpegCapture::decode_frame(AVFormatContext *format_ctx, AVCo
 	*audio_pts = -1;
 	bool has_audio = false;
 	do {
-		AVPacket pkt;
-		unique_ptr<AVPacket, decltype(av_packet_unref)*> pkt_cleanup(
-			&pkt, av_packet_unref);
-		av_init_packet(&pkt);
-		pkt.data = nullptr;
-		pkt.size = 0;
-		if (av_read_frame(format_ctx, &pkt) == 0) {
-			if (pkt.stream_index == audio_stream_index && audio_callback != nullptr) {
-				audio_callback(&pkt, format_ctx->streams[audio_stream_index]->time_base);
+		AVPacketWithDeleter pkt = av_packet_alloc_unique();
+		pkt->data = nullptr;
+		pkt->size = 0;
+		if (av_read_frame(format_ctx, pkt.get()) == 0) {
+			if (pkt->stream_index == audio_stream_index && audio_callback != nullptr) {
+				audio_callback(pkt.get(), format_ctx->streams[audio_stream_index]->time_base);
 			}
-			if (pkt.stream_index == video_stream_index && video_callback != nullptr) {
-				video_callback(&pkt, format_ctx->streams[video_stream_index]->time_base);
+			if (pkt->stream_index == video_stream_index && video_callback != nullptr) {
+				video_callback(pkt.get(), format_ctx->streams[video_stream_index]->time_base);
 			}
-			if (pkt.stream_index == video_stream_index && global_flags.transcode_video) {
-				if (avcodec_send_packet(video_codec_ctx, &pkt) < 0) {
+			if (pkt->stream_index == video_stream_index && global_flags.transcode_video) {
+				if (avcodec_send_packet(video_codec_ctx, pkt.get()) < 0) {
 					fprintf(stderr, "%s: Cannot send packet to video codec.\n", pathname.c_str());
 					*error = true;
 					return AVFrameWithDeleter(nullptr);
 				}
-			} else if (pkt.stream_index == audio_stream_index && global_flags.transcode_audio) {
+			} else if (pkt->stream_index == audio_stream_index && global_flags.transcode_audio) {
 				has_audio = true;
-				if (avcodec_send_packet(audio_codec_ctx, &pkt) < 0) {
+				if (avcodec_send_packet(audio_codec_ctx, pkt.get()) < 0) {
 					fprintf(stderr, "%s: Cannot send packet to audio codec.\n", pathname.c_str());
 					*error = true;
 					return AVFrameWithDeleter(nullptr);
 				}
-			} else if (pkt.stream_index == subtitle_stream_index) {
-				last_subtitle = string(reinterpret_cast<const char *>(pkt.data), pkt.size);
+			} else if (pkt->stream_index == subtitle_stream_index) {
+				last_subtitle = string(reinterpret_cast<const char *>(pkt->data), pkt->size);
 				has_last_subtitle = true;
 			}
 		} else {