> -----Original Message-----
> From: ffmpeg-devel <ffmpeg-devel-boun...@ffmpeg.org> On Behalf Of Martin
> Storsjö
> Sent: Monday, May 23, 2022 12:58 PM
> To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
> Subject: Re: [FFmpeg-devel] [PATCH 1/3] fftools: Stop using av_fopen_utf8
> 
> On Mon, 23 May 2022, Soft Works wrote:
> 
> >
> >
> >> -----Original Message-----
> >> From: ffmpeg-devel <ffmpeg-devel-boun...@ffmpeg.org> On Behalf Of
> Martin
> >> Storsjö
> >> Sent: Monday, May 23, 2022 12:53 PM
> >> To: FFmpeg development discussions and patches <ffmpeg-
> de...@ffmpeg.org>
> >> Subject: Re: [FFmpeg-devel] [PATCH 1/3] fftools: Stop using
> av_fopen_utf8
> >>
> >> On Sat, 21 May 2022, Soft Works wrote:
> >>
> >>> LGTM. (all three)
> >>>
> >>> Tested with VS project build (full static linkage, though).
> >>
> >> I discussed this with Anton on irc, and he was ok with the patchset
> too,
> >> so I pushed it now.
> >>
> >> // Martin
> >
> > Great, thanks.
> >
> > Shall I update mine now to cover the remaining fopen() calls?
> 
> Yep, that sounds good to me. It should be easier to move forward with the
> uncontroversial parts of the patchsets now.
> 
> // Martin

Sure, will do. 

I think I have addressed all concerns from the side of nil-admirari;
the last point was the mapping of the stat() function, which -
even though it works - could use a few more eyes taking a look at.

Any other specific concerns from your side?

Thanks,
sw







_______________________________________________
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