reflect/protoreflect: drop the ProtoEnum type
Drop the protoreflect.ProtoEnum type (containing a single method
returning a protoreflect.Enum) and make generated enum types
directly implement protoreflect.Enum instead.
Messages have a two-level type split (ProtoMessage and Message) to
minimize conflicts between reflection methods and field names. Enums
need no such split, since enums do not have fields and therefore have
no source of conflicts.
Change-Id: I2b6222e9404253e6bfef2217859e1b760ffcd29b
Reviewed-on: https://go-review.googlesource.com/c/156902
Reviewed-by: Joe Tsai <thebrokentoaster@gmail.com>
Reviewed-by: Damien Neil <dneil@google.com>
diff --git a/internal/impl/message_test.go b/internal/impl/message_test.go
index 97fbabd..b98fe65 100644
--- a/internal/impl/message_test.go
+++ b/internal/impl/message_test.go
@@ -936,7 +936,7 @@
FullName: "EnumProto2",
Values: []ptype.EnumValue{{Name: "DEAD", Number: 0xdead}, {Name: "BEEF", Number: 0xbeef}},
}),
- func(_ pref.EnumType, n pref.EnumNumber) pref.ProtoEnum {
+ func(_ pref.EnumType, n pref.EnumNumber) pref.Enum {
return EnumProto2(n)
},
)
@@ -944,7 +944,6 @@
func (e EnumProto2) Enum() *EnumProto2 { return &e }
func (e EnumProto2) Type() pref.EnumType { return enumProto2Type }
func (e EnumProto2) Number() pref.EnumNumber { return pref.EnumNumber(e) }
-func (e EnumProto2) ProtoReflect() pref.Enum { return e }
type EnumProto3 int32
@@ -954,7 +953,7 @@
FullName: "EnumProto3",
Values: []ptype.EnumValue{{Name: "ALPHA", Number: 0}, {Name: "BRAVO", Number: 1}},
}),
- func(_ pref.EnumType, n pref.EnumNumber) pref.ProtoEnum {
+ func(_ pref.EnumType, n pref.EnumNumber) pref.Enum {
return EnumProto3(n)
},
)
@@ -962,7 +961,6 @@
func (e EnumProto3) Enum() *EnumProto3 { return &e }
func (e EnumProto3) Type() pref.EnumType { return enumProto3Type }
func (e EnumProto3) Number() pref.EnumNumber { return pref.EnumNumber(e) }
-func (e EnumProto3) ProtoReflect() pref.Enum { return e }
type EnumMessages struct {
EnumP2 *EnumProto2 `protobuf:"1"`