[FFmpeg-devel] [PATCH] configure: Remove a leftover comment about MSVC C99 support

Martin Storsjö martin at martin.st
Thu Oct 27 13:17:48 EEST 2022


On Wed, 19 Oct 2022, Martin Storsjö wrote:

> Support for building with older versions of MSVC (with the
> c99wrap/c99conv frontend) was removed in
> ce943dd6acbfdfc40223c0fb24d4cad438e6499c.
>
> Signed-off-by: Martin Storsjö <martin at martin.st>
> ---
> configure | 6 ------
> 1 file changed, 6 deletions(-)
>
> diff --git a/configure b/configure
> index 6712d045d9..ed52212f93 100755
> --- a/configure
> +++ b/configure
> @@ -4368,12 +4368,6 @@ case "$toolchain" in
>         esac
>     ;;
>     msvc)
> -        # Check whether the current MSVC version needs the C99 converter.
> -        # From MSVC 2013 (compiler major version 18) onwards, it does actually
> -        # support enough of C99 to build ffmpeg. Default to the new
> -        # behaviour if the regexp was unable to match anything, since this
> -        # successfully parses the version number of existing supported
> -        # versions that require the converter (MSVC 2010 and 2012).
>         cl_major_ver=$(cl.exe 2>&1 | sed -n 's/.*Version \([[:digit:]]\{1,\}\)\..*/\1/p')
>         if [ -z "$cl_major_ver" ] || [ $cl_major_ver -ge 18 ]; then
>             cc_default="cl.exe"
> -- 
> 2.37.0 (Apple Git-136)

Will push later today.

// Martin


More information about the ffmpeg-devel mailing list