During the discussion of 
[JDK-8292981](https://bugs.openjdk.org/browse/JDK-8292981) an opinion was 
voiced that we should stop using INTPTR_FORMAT when printing pointers.

Some background that could explain why some tend to use INTPTR_FORMAT instead 
of PTR_FORMAT:

Both those format specifiers require an integer and the compiler barfs if you 
send in a pointer. We therefore have a utility function named p2i, which 
converts the pointer to an integer. So, everywhere we have to write print line 
like this: `print("my pointer: " PTR_FORMAT, p2i(my_pointer));`. Now, p2i 
returns an intptr_t, and it becomes natural for some to consider the type of 
the converted value (that we need because of the mentioned workaround), instead 
of the original type of the value.

With this enhancement I'd like to clean up the code that I often work in, so 
that it uses PTR_FORMAT when printing pointers. I'm leaving the rest of the 
code base for others to consider cleaning up.

Cleanups have been done in directories gc/, utilities/, memory/, and oops/.

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

Commit messages:
 - Fix oops/
 - Fix memory/
 - Fix utilities/
 - Fix gc/
 - Add SIZE_FORMAT_X_0

Changes: https://git.openjdk.org/jdk/pull/10141/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=10141&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8293304
  Stats: 125 lines in 39 files changed: 6 ins; 0 del; 119 mod
  Patch: https://git.openjdk.org/jdk/pull/10141.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/10141/head:pull/10141

PR: https://git.openjdk.org/jdk/pull/10141

Reply via email to