code review 5286051: [release-branch.r60] reflect: disallow Interface method...

[release-branch.r60] reflect: disallow Interface method on Value obtained via unexported name

exp/datafmt is incompatible with this change, so delete it.
It was scheduled for deletion soon anyway.

««« CL 5266054 / f798c2579fbd
reflect: disallow Interface method on Value obtained via unexported name

Had been allowing it for use by fmt, but it is too hard to lock down.
Fix other packages not to depend on it.

R=r, r
CC=golang-dev
http://codereview.appspot.com/5266054
»»»

[rsc, 2011-10-17 22:57:24]

Hello golang-dev@googlegroups.com,

I'd like you to review this change to
https://go.googlecode.com/hg/

[rsc, 2011-10-17 22:58:48]

Two notes:

1. Applied the patch manually (using hg patch) because it
needed some fuzz processing due to other changes in
all_test.go.

2. Deleted exp/datafmt in this CL too, rather than try to
keep it up to date.  I am confident that no one is using
exp/datafmt.

Russ

[r, 2011-10-17 22:59:00]

LGTM

[rsc, 2011-10-17 23:01:31]

*** Submitted as http://code.google.com/p/go/source/detail?r=01fa62f5e4e5 ***

[release-branch.r60] reflect: disallow Interface method on Value obtained via
unexported name

exp/datafmt is incompatible with this change, so delete it.
It was scheduled for deletion soon anyway.

««« CL 5266054 / f798c2579fbd
reflect: disallow Interface method on Value obtained via unexported name

Had been allowing it for use by fmt, but it is too hard to lock down.
Fix other packages not to depend on it.

R=r, r
CC=golang-dev
http://codereview.appspot.com/5266054
»»»

R=golang-dev, r
CC=golang-dev
http://codereview.appspot.com/5286051