Re: [FFmpeg-devel] [PATCH 1/3] avformat/avisynth: add missing avs_release_video_frame

From: Stephen Hutchinson <qyot27@gmail.com>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Subject: Re: [FFmpeg-devel] [PATCH 1/3] avformat/avisynth: add missing avs_release_video_frame
Date: Fri, 12 Aug 2022 17:25:10 -0400
Message-ID: <fea7f050-5865-3b34-7b56-dc2254fa86ee@gmail.com> (raw)
In-Reply-To: <20220808012532.41519-2-qyot27@gmail.com>

On 8/7/22 9:25 PM, Stephen Hutchinson wrote:
> The AviSynth C API requires using avs_release_video_frame
> whenever avs_get_frame has been used, but the recent addition
> of frameprop reading to the demuxer was missing this in
> avisynth_create_stream_video.
> 
> ---
>   libavformat/avisynth.c | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/libavformat/avisynth.c b/libavformat/avisynth.c
> index a97d12b6b6..98b4d68a57 100644
> --- a/libavformat/avisynth.c
> +++ b/libavformat/avisynth.c
> @@ -728,6 +728,7 @@ static int avisynth_create_stream_video(AVFormatContext *s, AVStream *st)
>                   st->codecpar->chroma_location = AVCHROMA_LOC_UNSPECIFIED;
>               }
>           }
> +        avs_library.avs_release_video_frame(frame);
>       } else {
>           st->codecpar->field_order = AV_FIELD_UNKNOWN;
>           /* AviSynth works with frame-based video, detecting field order can

Pushed.
_______________________________________________
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".