From: David Crawshaw <crawshaw@golang.org>
Date: Fri, 11 Nov 2016 00:02:07 +0000 (-0500)
Subject: reflect: rename, document TestUnaddressableField
X-Git-Tag: go1.8beta1~211
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=66477ec8307e18f751996d92ac8741596a23615a;p=gostls13.git

reflect: rename, document TestUnaddressableField

Change-Id: I94e0f3e4bccd44a67934ddb4d5fc7da57bb8ac9f
Reviewed-on: https://go-review.googlesource.com/33112
Reviewed-by: Russ Cox <rsc@golang.org>
Reviewed-by: Rob Pike <r@golang.org>
---

diff --git a/src/reflect/all_test.go b/src/reflect/all_test.go
index 4c11609218..7dfdfd8709 100644
--- a/src/reflect/all_test.go
+++ b/src/reflect/all_test.go
@@ -5914,8 +5914,13 @@ func TestSwapper(t *testing.T) {
 	}
 }
 
-func TestInaccessibleField(t *testing.T) {
-	var b Buffer
+// TestUnaddressableField tests that the reflect package will not allow
+// a type from another package to be used as a named type with an
+// unexported field.
+//
+// This ensures that unexported fields cannot be modified by other packages.
+func TestUnaddressableField(t *testing.T) {
+	var b Buffer // type defined in reflect, a different package
 	var localBuffer struct {
 		buf []byte
 	}