fix up %p
- use an interface {Get()}
- implement Get for maps, slices
- for slices, retrieves the address of the end of the array, which will give the
same value for every slice of the same array.
R=rsc
CC=golang-dev
https://golang.org/cl/179129
diff --git a/src/pkg/fmt/print.go b/src/pkg/fmt/print.go
index 044ac17..cc2c82c 100644
--- a/src/pkg/fmt/print.go
+++ b/src/pkg/fmt/print.go
@@ -136,6 +136,12 @@
GoString() string
}
+// getter is implemented by any value that has a Get() method,
+// which means the object contains a pointer. Used by %p.
+type getter interface {
+ Get() uintptr
+}
+
const allocSize = 32
type pp struct {
@@ -803,12 +809,9 @@
// pointer, including addresses of reference types.
case 'p':
switch v := field.(type) {
- case *reflect.PtrValue:
+ case getter:
p.fmt.fmt_s("0x")
p.fmt.fmt_uX64(uint64(v.Get()))
- case *reflect.ChanValue, *reflect.MapValue, *reflect.SliceValue:
- p.fmt.fmt_s("0x")
- p.fmt.fmt_uX64(uint64(field.Addr()))
default:
goto badtype
}
diff --git a/src/pkg/reflect/value.go b/src/pkg/reflect/value.go
index bbc66de..3c77b87 100644
--- a/src/pkg/reflect/value.go
+++ b/src/pkg/reflect/value.go
@@ -567,6 +567,14 @@
// Set sets v to the value x.
func (v *SliceValue) SetValue(x Value) { v.Set(x.(*SliceValue)) }
+// Get returns the uintptr address of the v.Cap()'th element. This gives
+// the same result for all slices of the same array.
+// It is mainly useful for printing.
+func (v *SliceValue) Get() uintptr {
+ typ := v.typ.(*SliceType)
+ return uintptr(v.addr()) + uintptr(v.Cap())*typ.Elem().Size()
+}
+
// Slice returns a sub-slice of the slice v.
func (v *SliceValue) Slice(beg, end int) *SliceValue {
cap := v.Cap()
@@ -970,6 +978,10 @@
// Set sets v to the value x.
func (v *MapValue) SetValue(x Value) { v.Set(x.(*MapValue)) }
+// Get returns the uintptr value of v.
+// It is mainly useful for printing.
+func (v *MapValue) Get() uintptr { return *(*uintptr)(v.addr) }
+
// implemented in ../pkg/runtime/reflect.cgo
func mapaccess(m, key, val *byte) bool
func mapassign(m, key, val *byte)