all: drop reflect/prototype package
Remove the remaining uses of the prototype package.
The most significant change is to impl.MessageInfo, which now directly
implements the MessageType interface. This involves two notable changes
to exported fields of MessageInfo:
- PBType is now Desc.
- GoType is now GoReflectType. (Name changed to avoid a conflict with
the GoType method of the MessageType interface.)
Fixes golang/protobuf#911
Change-Id: Ie2aa4766d6887ceaa9cf06b1f109aa6e6e2a208f
Reviewed-on: https://go-review.googlesource.com/c/protobuf/+/189340
Reviewed-by: Joe Tsai <thebrokentoaster@gmail.com>
diff --git a/internal/impl/codec_field.go b/internal/impl/codec_field.go
index 9d587d2..16873e4 100644
--- a/internal/impl/codec_field.go
+++ b/internal/impl/codec_field.go
@@ -138,7 +138,7 @@
return 0, wire.ParseError(n)
}
if p.Elem().IsNil() {
- p.SetPointer(pointerOfValue(reflect.New(mi.GoType.Elem())))
+ p.SetPointer(pointerOfValue(reflect.New(mi.GoReflectType.Elem())))
}
if _, err := mi.unmarshalPointer(v, p.Elem(), 0, opts); err != nil {
return 0, err
@@ -256,7 +256,7 @@
return 0, errUnknown
}
if p.Elem().IsNil() {
- p.SetPointer(pointerOfValue(reflect.New(mi.GoType.Elem())))
+ p.SetPointer(pointerOfValue(reflect.New(mi.GoReflectType.Elem())))
}
return mi.unmarshalPointer(b, p.Elem(), num, opts)
}
@@ -367,7 +367,7 @@
if n < 0 {
return 0, wire.ParseError(n)
}
- m := reflect.New(mi.GoType.Elem()).Interface()
+ m := reflect.New(mi.GoReflectType.Elem()).Interface()
mp := pointerOfIface(m)
if _, err := mi.unmarshalPointer(v, mp, 0, opts); err != nil {
return 0, err
@@ -571,7 +571,7 @@
if wtyp != wire.StartGroupType {
return 0, errUnknown
}
- m := reflect.New(mi.GoType.Elem()).Interface()
+ m := reflect.New(mi.GoReflectType.Elem()).Interface()
mp := pointerOfIface(m)
n, err := mi.unmarshalPointer(b, mp, num, opts)
if err != nil {