>> 
>> The patch also did not update the debugging message in
>> C_set_or_change_heap_size.
>> -    C_dbg(C_text("debug"), C_text("heap resized to %d bytes\n"), (int)heap);
>> +    C_dbg(C_text("debug"), C_text("heap resized to %zu bytes\n"), heap);
>> %zu is a C99-ism, I don't know if people are ok with that.
> 
> So far, generally we've avoided these because they're not very portable.
> We don't really have an official policy on this, but I guess if we're
> going to use size_t consistently, we should use the matching printf
> specifiers too.

This might be a problem on Windows. In general, format-specifiers have driven
me mad more than once, due to portability issues.


cheers,
felix

_______________________________________________
Chicken-hackers mailing list
Chicken-hackers@nongnu.org
https://lists.nongnu.org/mailman/listinfo/chicken-hackers

Reply via email to