On Fri, 13 Dec 2013 14:30:21 -0500
Rafaël Carré <[email protected]> wrote:

> printf is an alias for gnu_printf everywhere, except when targetting
> windows where it is an alias for ms_printf.
> 
> We do want the posix versions and not the MS versions, that's why
> we define __USE_MINGW_ANSI_STDIO to 1 elsewhere.
> ---
>  libavutil/attributes.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavutil/attributes.h b/libavutil/attributes.h
> index d7f2bb5..712c44a 100644
> --- a/libavutil/attributes.h
> +++ b/libavutil/attributes.h
> @@ -111,7 +111,7 @@
>  
>  #ifdef __GNUC__
>  #    define av_builtin_constant_p __builtin_constant_p
> -#    define av_printf_format(fmtpos, attrpos) 
> __attribute__((__format__(__printf__, fmtpos, attrpos)))
> +#    define av_printf_format(fmtpos, attrpos) 
> __attribute__((__format__(__gnu_printf__, fmtpos, attrpos)))
>  #else
>  #    define av_builtin_constant_p(x) 0
>  #    define av_printf_format(fmtpos, attrpos)

As far as I understand, __printf__ is mapped to ms_printf on MinGW,
even if printf() maps to the C99 replacement. Shouldn't this be fixed
upstream so that __printf__ always correspondents to the current
printf() implementation?
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to