On Mon, 9 May 2022, softworkz wrote:

From: softworkz <softwo...@hotmail.com>

Signed-off-by: softworkz <softwo...@hotmail.com>
---
   use av_fopen_utf8() instead of plain fopen()

   Unify file access operations by replacing usages of direct calls to
   posix fopen()

   v2: Remove changes to fftools for now

Published-As: 
https://github.com/ffstaging/FFmpeg/releases/tag/pr-ffstaging-26%2Fsoftworkz%2Fsubmit_replace_fopen-v2
Fetch-It-Via: git fetch https://github.com/ffstaging/FFmpeg 
pr-ffstaging-26/softworkz/submit_replace_fopen-v2
Pull-Request: https://github.com/ffstaging/FFmpeg/pull/26

Range-diff vs v1:

1:  5802c8526c < -:  ---------- fftools: use av_fopen_utf8() instead of plain 
fopen()
2:  3266640a93 = 1:  e47287be64 avfilter: use av_fopen_utf8() instead of plain 
fopen()


libavfilter/af_firequalizer.c | 2 +-
libavfilter/vf_deshake.c      | 2 +-
libavfilter/vf_signature.c    | 4 ++--
libavfilter/vf_ssim.c         | 2 +-
libavfilter/vf_vmafmotion.c   | 2 +-
5 files changed, 6 insertions(+), 6 deletions(-)

LGTM I think. For fully fixing the situation about this function, I believe we're going to need to rename it (as the proper solution won't be a public function), but as there's already some uses, it's probably fine to first take it into use consistently, and then rename all the occurrances later.

But we should probably add a copy of file_open.o in libavfilter too (as you noted). This is indeed a preexisting problem, but the issue will become more visible if we use it in more places.

// Martin

_______________________________________________
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