[FFmpeg-devel] [PATCH v4 0/2] use av_fopen_utf8() instead of plain fopen()

[Soft Works]

> -----Original Message-----
> From: Tobias Rapp <t.rapp at noa-archive.com>
> Sent: Wednesday, May 18, 2022 9:01 AM
> To: ffmpegagent <ffmpegagent at gmail.com>; ffmpeg-devel at ffmpeg.org
> Cc: Martin Storsjö <martin at martin.st>; Soft Works
> <softworkz at hotmail.com>
> Subject: Re: [PATCH v4 0/2] use av_fopen_utf8() instead of plain
> fopen()
> 
> On 17/05/2022 14:29, ffmpegagent wrote:
> > Unify file access operations by replacing usages of direct calls to
> posix
> > fopen()
> >
> > v2: Remove changes to fftools for now
> > v3: Add some additional replacements
> > v4: Fix and improve commit messages
> >
> > softworkz (2):
> >    avfilter: use av_fopen_utf8() instead of plain fopen()
> >    avcodec/dvdsubdec: use av_fopen_utf8() instead of plain fopen()
> >
> >   libavcodec/dvdsubdec.c            | 2 +-
> >   libavfilter/af_firequalizer.c     | 2 +-
> >   libavfilter/vf_deshake.c          | 2 +-
> >   libavfilter/vf_psnr.c             | 2 +-
> >   libavfilter/vf_signature.c        | 4 ++--
> >   libavfilter/vf_ssim.c             | 2 +-
> >   libavfilter/vf_vidstabdetect.c    | 2 +-
> >   libavfilter/vf_vidstabtransform.c | 2 +-
> >   libavfilter/vf_vmafmotion.c       | 2 +-
> >   9 files changed, 10 insertions(+), 10 deletions(-)
> >
> > [...]
> 
> Commit messages look fine to me now. I will leave the decision to
> others
> about the order of changes -- whether this patch-set comes first, or
> the
> fix for the CRT linking issue (possibly replacing this public function
> with a private copy).

To eliminate the question, I'm including that change in
the v5 of this patchset.

Thanks,
softworkz