RFR: 8346990: Remove INTX_FORMAT and UINTX_FORMAT macros

Julian Waters jwaters at openjdk.org
Fri Jan 3 15:43:41 UTC 2025


On Fri, 3 Jan 2025 14:32:39 GMT, Coleen Phillimore <coleenp at 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


More information about the shenandoah-dev mailing list