From bddfc337f9b053740b51e1fd8429f84dafa89205 Mon Sep 17 00:00:00 2001 From: David Crawshaw Date: Wed, 27 Apr 2016 13:10:49 -0400 Subject: [PATCH] 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 --- src/reflect/all_test.go | 3 +++ src/reflect/type.go | 1 + 2 files changed, 4 insertions(+) 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 -- 2.48.1