[Mesa-dev] [PATCH] util: Only use open coded snprintf for MSVC.

Brian Paul brianp at vmware.com
Wed Nov 14 07:05:34 PST 2012


On 11/14/2012 12:16 AM, Vinson Lee wrote:
> MinGW has snprintf.
>
> The patch fixes these warnings with the MinGW SCons build.
>
> src/gallium/auxiliary/util/u_snprintf.c:459:1: warning: no previous prototype for ‘util_vsnprintf’ [-Wmissing-prototypes]
> src/gallium/auxiliary/util/u_snprintf.c:1436:1: warning: no previous prototype for ‘util_snprintf’ [-Wmissing-prototypes]
>
> Signed-off-by: Vinson Lee<vlee at freedesktop.org>
> ---
>   src/gallium/auxiliary/util/u_snprintf.c |    2 +-
>   src/gallium/auxiliary/util/u_string.h   |    4 ++--
>   2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/src/gallium/auxiliary/util/u_snprintf.c b/src/gallium/auxiliary/util/u_snprintf.c
> index e16f103..a24b6ff 100644
> --- a/src/gallium/auxiliary/util/u_snprintf.c
> +++ b/src/gallium/auxiliary/util/u_snprintf.c
> @@ -167,7 +167,7 @@
>   #if HAVE_CONFIG_H
>   #include<config.h>
>   #else
> -#ifdef _WIN32
> +#ifdef _MSC_VER
>   #define vsnprintf util_vsnprintf
>   #define snprintf util_snprintf
>   #define HAVE_VSNPRINTF 0
> diff --git a/src/gallium/auxiliary/util/u_string.h b/src/gallium/auxiliary/util/u_string.h
> index 15630ad..3d5aba5 100644
> --- a/src/gallium/auxiliary/util/u_string.h
> +++ b/src/gallium/auxiliary/util/u_string.h
> @@ -35,7 +35,7 @@
>   #ifndef U_STRING_H_
>   #define U_STRING_H_
>
> -#if !defined(_WIN32)&&  !defined(XF86_LIBC_H)
> +#if !defined(_MSC_VER)

Are you sure that shouldn't be:

#if !defined(_MSC_VER) && !defined(XF86_LIBC_H)



>   #include<stdio.h>
>   #endif
>   #include<stddef.h>
> @@ -64,7 +64,7 @@ util_strchrnul(const char *s, char c)
>
>   #endif
>
> -#ifdef _WIN32
> +#ifdef _MSC_VER
>
>   int util_vsnprintf(char *, size_t, const char *, va_list);
>   int util_snprintf(char *str, size_t size, const char *format, ...);



More information about the mesa-dev mailing list