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