[FFmpeg-devel] [PATCH v2 2/3] tests/source-check: Fix make inclusion-guard check EOL-agnostic
Andreas Rheinhardt
andreas.rheinhardt at outlook.com
Thu May 22 13:41:39 EEST 2025
softworkz:
> From: softworkz <softworkz at hotmail.com>
>
> ..to make it work when checked out with autocrlf=on,
> which is Git default on Windows.
>
> Signed-off-by: softworkz <softworkz at hotmail.com>
> ---
> tests/fate/source-check.sh | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/fate/source-check.sh b/tests/fate/source-check.sh
> index 4d7e175784..99e869e869 100755
> --- a/tests/fate/source-check.sh
> +++ b/tests/fate/source-check.sh
> @@ -28,7 +28,7 @@ for f in `git ls-files | grep '\.h$'` ; do
> -e 's/_vaf_/_/' \
> | tr abcdefghijklmnopqrstuvwxyz ABCDEFGHIJKLMNOPQRSTUVWXYZ`"
>
> - git grep -L "^#define $macro$" $f
> + git grep -L "^#define $macro\>" $f
This makes the test less strict. Why don't we instead just specify that
the repo should be checked out with lf only? Would this break something?
> done
>
> echo "Use of av_clip() where av_clip_uintp2() could be used:"
More information about the ffmpeg-devel
mailing list