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


// xufuji456


------------------ ???????? ------------------
??????:                                                                         
                                               "ffmpeg-devel"                   
                                                                 
<geo...@nsup.org&gt;;
????????:&nbsp;2023??3??3??(??????) ????5:31
??????:&nbsp;"FFmpeg development discussions and 
patches"<ffmpeg-devel@ffmpeg.org&gt;;
????:&nbsp;"??????"<839789...@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-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to