[PATCH] comctl32/listview: Fix printf format (PVS-Studio)

Alexandre Julliard julliard at winehq.org
Wed Oct 28 07:35:55 CDT 2015


Nikolay Sivov <nsivov at codeweavers.com> writes:

> @@ -587,7 +587,7 @@ static const char* debugscrollinfo(const SCROLLINFO *pScrollInfo)
>      int len, size = DEBUG_BUFFER_SIZE;
>  
>      if (pScrollInfo == NULL) return "(null)";
> -    len = snprintf(buf, size, "{cbSize=%d, ", pScrollInfo->cbSize);
> +    len = snprintf(buf, size, "{cbSize=%u, ", pScrollInfo->cbSize);

That doesn't seem very necessary. I don't mind a few fixes like that,
but I'm concerned that there would be a million of them. How many do you
expect?

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list