<div><br></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue 1. Aug 2023 at 18:32, Coleen Phillimore <<a href="mailto:coleenp@openjdk.org">coleenp@openjdk.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)">On Tue, 1 Aug 2023 13:16:17 GMT, Kim Barrett <<a href="mailto:kbarrett@openjdk.org" target="_blank">kbarrett@openjdk.org</a>> wrote:<br>
<br>
>> src/hotspot/os/windows/os_windows.cpp line 3382:<br>
>> <br>
>>> 3380:     if (Verbose && PrintMiscellaneous) {<br>
>>> 3381:       reserveTimer.stop();<br>
>>> 3382:       tty->print_cr("reserve_memory of %zx bytes took " JLONG_FORMAT " ms (" JLONG_FORMAT " ticks)", bytes,<br>
>> <br>
>> SIZE_FORMAT.<br>
><br>
> Regarding SIZE_FORMAT - <a href="https://bugs.openjdk.org/browse/JDK-8256379" rel="noreferrer" target="_blank">https://bugs.openjdk.org/browse/JDK-8256379</a>.  Some people have been replacing<br>
> uses of SIZE_FORMAT when touching the relevant code for other reasons.<br>
<br>
Yes, I think it might be best if people started using %zd rather than adding new SIZE_FORMAT to the code. I haven't figured out the best way to change the rest.  If using %zx, I suggest adding 0x%zx.</blockquote><div dir="auto"><br></div><div dir="auto">Pity, I liked those macros. We also lose the ability to compile with older compilers, e.g. vs2013.</div><div dir="auto"><br></div><div dir="auto">But I really dislike wholesale replacements of X with X prime (newer, shinier) for aesthetic reasons. I much rather have those replacements we agree upon be done when code is touched for other reasons.  That causes much less overhead for update maintainers.</div><div dir="auto"><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;padding-left:1ex;border-left-color:rgb(204,204,204)" dir="auto">-------------<br>
<br>
PR Review Comment: <a href="https://git.openjdk.org/jdk/pull/15063#discussion_r1280885515" rel="noreferrer" target="_blank">https://git.openjdk.org/jdk/pull/15063#discussion_r1280885515</a><br>
</blockquote></div></div>