[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Chicken-hackers] rounding off the heap size_t change to allow initi
From: |
Felix |
Subject: |
Re: [Chicken-hackers] rounding off the heap size_t change to allow initial large heaps |
Date: |
Fri, 29 Mar 2013 21:27:25 +0100 (CET) |
>>
>> 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