]> Cypherpunks repositories - gostls13.git/commit
fmt: treat reflect.Value specially - as the value it holds
authorRob Pike <r@golang.org>
Thu, 9 Apr 2015 23:30:48 +0000 (16:30 -0700)
committerRob Pike <r@golang.org>
Wed, 15 Apr 2015 15:59:39 +0000 (15:59 +0000)
commit049b89dc6f6b6f1001672dd5456197b74a97cbec
tree597d66fd7a9382c9c91890c8fed3cef21713e008
parent2c89992f445a631da250517d6f9b9fcd7852872e
fmt: treat reflect.Value specially - as the value it holds

When a reflect.Value is passed to Printf (etc.), fmt called the
String method, which does not disclose its contents. To get the
contents, one could call Value.Interface(), but that is illegal
if the Value is not exported or otherwise forbidden.

This CL improves the situation with a trivial change to the
fmt package: when we see a reflect.Value as an argument,
we treat it exactly as we treat a reflect.Value we make inside
the package. This means that we always print the
contents of the Value as if _that_ was the argument to Printf.

This is arguably a breaking change but I think it is a genuine
improvement and no greater a break than many other tweaks
we have made to formatted output from this package.

Fixes #8965.

Change-Id: Ifc2a4ce3c1134ad5160e101d2196c22f1542faab
Reviewed-on: https://go-review.googlesource.com/8731
Reviewed-by: roger peppe <rogpeppe@gmail.com>
Reviewed-by: Russ Cox <rsc@golang.org>
src/fmt/doc.go
src/fmt/fmt_test.go
src/fmt/print.go
src/reflect/value.go