Thank you for your reminding, George.
I really ignore av_freep() above.
Please ignore this patch.


// xufuji456


------------------ 原始邮件 ------------------
发件人:                                                                                                                        "ffmpeg-devel"                                                                                    <george@nsup.org&gt;;
发送时间:&nbsp;2023年3月3日(星期五) 下午5:31
收件人:&nbsp;"FFmpeg development discussions and patches"<ffmpeg-devel@ffmpeg.org&gt;;
抄送:&nbsp;"徐福隆"<839789740@qq.com&gt;;
主题:&nbsp;Re: [FFmpeg-devel] [PATCH] fftools/ffmpeg: reset pointer, which could be reused by Android and iOS



"zhilizhao(赵志立)" (12023-03-03):
&gt; &gt;&nbsp;&nbsp;&nbsp;&nbsp; av_freep(&amp;input_files);
&gt; &gt;&nbsp;&nbsp;&nbsp;&nbsp; av_freep(&amp;output_files);
&gt; &gt; +&nbsp;&nbsp;&nbsp; input_files&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; = NULL;
&gt; &gt; +&nbsp;&nbsp;&nbsp; output_files&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; = NULL;
&gt; Until we decided to make ffmpeg cmd works as a library,
&gt; it doesn’t matter.

It is worse than that: the patch completely ignores the semantic of
av_freep().

Useless patch with no explanations at all in the mail =&gt; just ignore the
mail.

&gt;		&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; I’m interested on the idea to make fftools
&gt; work like a library. But it depends on the community.

That would be more or less what an extended libavfilter with movie
source and sink would be.

Regards,

-- 
&nbsp; Nicolas George
_______________________________________________
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".