On 02.03.25 22:08, Thomas Munro wrote:
And one more thing like that: in a couple of places we see warnings on macOS CI that I'd missed: when printing the result of i64abs() as PRId64, because it happens to use labs() and it happens to define int64_t as long long, and when printing a Datum as PRIx64, because Datum is uintptr_t and it happens to define that as unsigned long. I suppose we should cast to int64 in the definition of c.h's i64abs() macro and a couple of similar things,
agreed
and cast Datum to uint64 in that one place that wants to print it out.
Since Datum is uintptr_t, it should be printed using the format PRIxPTR. Then it should work out.
