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