From: David Crawshaw Date: Wed, 27 Apr 2016 17:10:49 +0000 (-0400) Subject: reflect: fix strings of SliceOf-created types X-Git-Tag: go1.7beta1~453 X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=bddfc337f9b053740b51e1fd8429f84dafa89205;p=gostls13.git reflect: fix strings of SliceOf-created types The new type was inheriting the tflagExtraStar from its prototype. Fixes #15467 Change-Id: Ic22c2a55cee7580cb59228d52b97e1c0a1e60220 Reviewed-on: https://go-review.googlesource.com/22501 Reviewed-by: Ian Lance Taylor Run-TryBot: Ian Lance Taylor --- diff --git a/src/reflect/all_test.go b/src/reflect/all_test.go index 870ccbf521..f8ffaae8e1 100644 --- a/src/reflect/all_test.go +++ b/src/reflect/all_test.go @@ -3902,6 +3902,9 @@ func TestSliceOf(t *testing.T) { // check construction and use of type not in binary type T int st := SliceOf(TypeOf(T(1))) + if got, want := st.String(), "[]reflect_test.T"; got != want { + t.Errorf("SliceOf(T(1)).String()=%q, want %q", got, want) + } v := MakeSlice(st, 10, 10) runtime.GC() for i := 0; i < v.Len(); i++ { diff --git a/src/reflect/type.go b/src/reflect/type.go index 0213d56e83..2ceb3d3f66 100644 --- a/src/reflect/type.go +++ b/src/reflect/type.go @@ -2251,6 +2251,7 @@ func SliceOf(t Type) Type { prototype := *(**sliceType)(unsafe.Pointer(&islice)) slice := new(sliceType) *slice = *prototype + slice.tflag = 0 slice.str = resolveReflectName(newName(s, "", "", false)) slice.hash = fnv1(typ.hash, '[') slice.elem = typ