From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id 83C8F4565D for ; Mon, 13 Mar 2023 20:44:29 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 5F74968BC49; Mon, 13 Mar 2023 22:44:26 +0200 (EET) Received: from iq.passwd.hu (iq.passwd.hu [217.27.212.140]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id A43F168B8BE for ; Mon, 13 Mar 2023 22:44:20 +0200 (EET) Received: from localhost (localhost [127.0.0.1]) by iq.passwd.hu (Postfix) with ESMTP id 0E888E8733 for ; Mon, 13 Mar 2023 21:44:25 +0100 (CET) X-Virus-Scanned: amavisd-new at passwd.hu Received: from iq.passwd.hu ([127.0.0.1]) by localhost (iq.passwd.hu [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 2Q4t44K_dVDu for ; Mon, 13 Mar 2023 21:44:23 +0100 (CET) Received: from iq (iq [217.27.212.140]) by iq.passwd.hu (Postfix) with ESMTPS id 179EAE6BD9 for ; Mon, 13 Mar 2023 21:44:23 +0100 (CET) Date: Mon, 13 Mar 2023 21:44:23 +0100 (CET) From: Marton Balint To: FFmpeg development discussions and patches In-Reply-To: <20230310115635.13639-5-anton@khirnov.net> Message-ID: <3a6aa366-95f8-e89-6227-b954fbd02586@passwd.hu> References: <20230310115635.13639-1-anton@khirnov.net> <20230310115635.13639-5-anton@khirnov.net> MIME-Version: 1.0 Subject: Re: [FFmpeg-devel] [PATCH 5/6] fftools/ffplay: stop using AVFrame.pkt_pos X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: On Fri, 10 Mar 2023, Anton Khirnov wrote: > This field is ad-hoc and will be deprecated. Use the recently-added > AV_CODEC_FLAG_COPY_OPAQUE to pass arbitrary user data from packets to > frames. > --- > fftools/ffplay.c | 24 ++++++++++++++++++++++-- > 1 file changed, 22 insertions(+), 2 deletions(-) > > diff --git a/fftools/ffplay.c b/fftools/ffplay.c > index 0a96511475..d572eeabad 100644 > --- a/fftools/ffplay.c > +++ b/fftools/ffplay.c > @@ -146,6 +146,10 @@ typedef struct Clock { > int *queue_serial; /* pointer to the current packet queue serial, used for obsolete clock detection */ > } Clock; > > +typedef struct FrameData { > + int64_t pkt_pos; > +} FrameData; > + > /* Common struct for handling all types of decoded data and allocated render buffers. */ > typedef struct Frame { > AVFrame *frame; > @@ -640,6 +644,14 @@ static int decoder_decode_frame(Decoder *d, AVFrame *frame, AVSubtitle *sub) { > } > av_packet_unref(d->pkt); > } else { > + if (d->pkt->buf && !d->pkt->opaque_ref) { > + FrameData *fd; > + > + d->pkt->opaque_ref = av_buffer_allocz(sizeof(*fd)); Missing alloc check. Otherwise looks good. Thanks, Marton > + fd = (FrameData*)d->pkt->opaque_ref->data; > + fd->pkt_pos = d->pkt->pos; > + } > + > if (avcodec_send_packet(d->avctx, d->pkt) == AVERROR(EAGAIN)) { > av_log(d->avctx, AV_LOG_ERROR, "Receive_frame and send_packet both returned EAGAIN, which is an API violation.\n"); > d->packet_pending = 1; > @@ -2041,12 +2053,13 @@ static int audio_thread(void *arg) > goto the_end; > > while ((ret = av_buffersink_get_frame_flags(is->out_audio_filter, frame, 0)) >= 0) { > + FrameData *fd = frame->opaque_ref ? (FrameData*)frame->opaque_ref->data : NULL; > tb = av_buffersink_get_time_base(is->out_audio_filter); > if (!(af = frame_queue_peek_writable(&is->sampq))) > goto the_end; > > af->pts = (frame->pts == AV_NOPTS_VALUE) ? NAN : frame->pts * av_q2d(tb); > - af->pos = frame->pkt_pos; > + af->pos = fd ? fd->pkt_pos : -1; > af->serial = is->auddec.pkt_serial; > af->duration = av_q2d((AVRational){frame->nb_samples, frame->sample_rate}); > > @@ -2145,6 +2158,8 @@ static int video_thread(void *arg) > goto the_end; > > while (ret >= 0) { > + FrameData *fd; > + > is->frame_last_returned_time = av_gettime_relative() / 1000000.0; > > ret = av_buffersink_get_frame_flags(filt_out, frame, 0); > @@ -2155,13 +2170,15 @@ static int video_thread(void *arg) > break; > } > > + fd = frame->opaque_ref ? (FrameData*)frame->opaque_ref->data : NULL; > + > is->frame_last_filter_delay = av_gettime_relative() / 1000000.0 - is->frame_last_returned_time; > if (fabs(is->frame_last_filter_delay) > AV_NOSYNC_THRESHOLD / 10.0) > is->frame_last_filter_delay = 0; > tb = av_buffersink_get_time_base(filt_out); > duration = (frame_rate.num && frame_rate.den ? av_q2d((AVRational){frame_rate.den, frame_rate.num}) : 0); > pts = (frame->pts == AV_NOPTS_VALUE) ? NAN : frame->pts * av_q2d(tb); > - ret = queue_picture(is, frame, pts, duration, frame->pkt_pos, is->viddec.pkt_serial); > + ret = queue_picture(is, frame, pts, duration, fd ? fd->pkt_pos : -1, is->viddec.pkt_serial); > av_frame_unref(frame); > if (is->videoq.serial != is->viddec.pkt_serial) > break; > @@ -2567,6 +2584,9 @@ static int stream_component_open(VideoState *is, int stream_index) > av_dict_set(&opts, "threads", "auto", 0); > if (stream_lowres) > av_dict_set_int(&opts, "lowres", stream_lowres, 0); > + > + av_dict_set(&opts, "flags", "+copy_opaque", AV_DICT_MULTIKEY); > + > if ((ret = avcodec_open2(avctx, codec, &opts)) < 0) { > goto fail; > } > -- > 2.39.1 > > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe". > _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".