]> Cypherpunks.ru repositories - gostls13.git/commit
runtime: print hex numbers with hex prefixes in traceback debug
authorRuss Cox <rsc@golang.org>
Wed, 27 Jan 2021 02:26:01 +0000 (21:26 -0500)
committerRuss Cox <rsc@golang.org>
Fri, 19 Feb 2021 00:03:44 +0000 (00:03 +0000)
commita1e9148e3dbb20a18e0139583e7d835cc7a820bf
tree9c1c01083d41ddd5dfd017c8a4697577b85f0bcb
parent75e273fc2c183896a11bf23f0688c38059933336
runtime: print hex numbers with hex prefixes in traceback debug

If traceback fails, it prints a helpful hex dump of the stack.
But the hex numbers have no 0x prefix, which might make it
a little unclear that they are hex.
We only print two per line, so there is plenty of room for the 0x.
Print it, which lets us delete a custom hex formatter.

Also, in the translated <name+off> hints, print off in hex
(with a 0x prefix). The offsets were previously decimal, which
could have been confused for hex since none of the hex had
0x prefixes. And decimal is kind of useless anyway since the
offsets shown in the main traceback are hex, so you can't
easily match them up without mental base conversions.

Just print hex everywhere, clearly marked by 0x.

This CL is part of a stack adding windows/arm64
support (#36439), intended to land in the Go 1.17 cycle.
This CL is, however, not windows/arm64-specific.
It is cleanup meant to make the port (and future ports) easier.

Change-Id: I72d26a4e41ada38b620bf8fe3576d787a2e59b47
Reviewed-on: https://go-review.googlesource.com/c/go/+/288809
Trust: Russ Cox <rsc@golang.org>
Reviewed-by: Cherry Zhang <cherryyz@google.com>
src/runtime/print.go