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/legacy/message.go b/internal/legacy/message.go
index f4b358f..5158374 100644
--- a/internal/legacy/message.go
+++ b/internal/legacy/message.go
@@ -219,8 +219,8 @@
// Populate EnumType and MessageType.
if f.EnumType == nil && f.Kind == pref.EnumKind {
- if ev, ok := reflect.Zero(t).Interface().(pref.ProtoEnum); ok {
- f.EnumType = ev.ProtoReflect().Type()
+ if ev, ok := reflect.Zero(t).Interface().(pref.Enum); ok {
+ f.EnumType = ev.Type()
} else {
f.EnumType = loadEnumDesc(t)
}