On Sun, 3 Oct 2021 18:58:50 GMT, Andrey Turbanov <d...@openjdk.java.net> wrote:
> Cleanup unnecessary toString() calls when conversion will happen implicitly
> anyway
Marked as reviewed by cjplummer (Reviewer).
src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/ui/classbrowser/HTMLGenerator.java
line 1012:
> 1010: tmpBuf.append(Long.toHexString(startPc +
> visitor.getInstructionSize()));
> 1011: tmpBuf.append(",0x");
> 1012: tmpBuf.append(Long.toHexString(startPc));
Overall these changes look good, although I do wonder what motivated to
`toString()` calls in the first place, especially this example where lines 1010
and 1012 are similar, but only 1010 used `toString()`.
-------------
PR: https://git.openjdk.java.net/jdk/pull/5800
