16.05.2021 04:30:06 Simon Burge <sim...@netbsd.org>:
> Hi Roland,
>
> "Roland Illig" wrote:
>
>> Module Name: xsrc
>> Committed By: rillig
>> Date: Sat May 15 19:30:15 UTC 2021
>>
>> Modified Files:
>>
>> xsrc/external/mit/xterm/dist: misc.c
>>
>> Log Message:
>>
>> xterm: fix Clang build on i386
>>
>> xsrc/external/mit/xterm/dist/misc.c:3250:47: error: format specifies
>> type 'unsigned long' but the argument has type 'size_t' (aka 'unsigned
>> int') [-Werror,-Wformat]
>
> - printf("color (ignored, length %lu)\n", have);
> + printf("color (ignored, length %lu)\n", (unsigned long)have);
>
> Would using "%zu" and reverting the cast be a better fix here?
I had considered this as well but found another instance in the same file where
a size_t was fed to printf by casting it to unsigned long, therefore I did it
this way, for consistency.
