On Fri, 3 Jan 2025 14:32:39 GMT, Coleen Phillimore <cole...@openjdk.org> wrote:

> There are a lot of format modifiers that are noisy and unnecessary in the 
> code.  This change removes the INTX variants.  It's not that disruptive even 
> for backporting because %z modifier has been available for a long time so 
> should backport fine.  This was mostly done with a sed script plus some hand 
> fixups.
> 
> Testing mach5 and other platform cross compilations in progress.  Opening 
> this for GHA testing.

Speaking of %z, there is a non Standard %Ix in os_windows.cpp


tty->print_cr("reserve_memory of %Ix bytes took " JLONG_FORMAT " ms (" 
JLONG_FORMAT " ticks)", bytes,
                    reserveTimer.milliseconds(), reserveTimer.ticks());


Could changing that to %zu be trivial enough to fit into this change?

-------------

PR Comment: https://git.openjdk.org/jdk/pull/22916#issuecomment-2569435948

Reply via email to