internal/impl: add ProtoMessageV1Of and ProtoMessageV2Of
These functions are used by v1 code to switch between
protoV1.Message and protoV2.Message.
Wrappers over them may be exposed in a hypothetical protoadapt package.
Change-Id: Ib5265420e34e9888b540307962c70189625d663c
Reviewed-on: https://go-review.googlesource.com/c/protobuf/+/193180
Reviewed-by: Damien Neil <dneil@google.com>
diff --git a/internal/impl/api_export.go b/internal/impl/api_export.go
index 8c35f8f..55c5481 100644
--- a/internal/impl/api_export.go
+++ b/internal/impl/api_export.go
@@ -5,11 +5,15 @@
package impl
import (
+ "fmt"
"reflect"
"strconv"
"google.golang.org/protobuf/encoding/prototext"
+ "google.golang.org/protobuf/proto"
+ "google.golang.org/protobuf/reflect/protoreflect"
pref "google.golang.org/protobuf/reflect/protoreflect"
+ piface "google.golang.org/protobuf/runtime/protoiface"
)
// Export is a zero-length named type that exists only to export a set of
@@ -28,14 +32,6 @@
return legacyWrapEnum(reflect.ValueOf(e))
}
-// EnumTypeOf returns the protoreflect.EnumType for e.
-func (Export) EnumTypeOf(e enum) pref.EnumType {
- if ev, ok := e.(pref.Enum); ok {
- return ev.Type()
- }
- return legacyLoadEnumType(reflect.TypeOf(e))
-}
-
// EnumDescriptorOf returns the protoreflect.EnumDescriptor for e.
func (Export) EnumDescriptorOf(e enum) pref.EnumDescriptor {
if ev, ok := e.(pref.Enum); ok {
@@ -44,6 +40,14 @@
return LegacyLoadEnumDesc(reflect.TypeOf(e))
}
+// EnumTypeOf returns the protoreflect.EnumType for e.
+func (Export) EnumTypeOf(e enum) pref.EnumType {
+ if ev, ok := e.(pref.Enum); ok {
+ return ev.Type()
+ }
+ return legacyLoadEnumType(reflect.TypeOf(e))
+}
+
// EnumStringOf returns the enum value as a string, either as the name if
// the number is resolvable, or the number formatted as a string.
func (Export) EnumStringOf(ed pref.EnumDescriptor, n pref.EnumNumber) string {
@@ -58,28 +62,81 @@
// and must be a pointer to a named struct type.
type message = interface{}
-// MessageOf returns the protoreflect.Message interface over m.
-func (Export) MessageOf(m message) pref.Message {
- if mv, ok := m.(pref.ProtoMessage); ok {
- return mv.ProtoReflect()
+// legacyMessageWrapper wraps a v2 message as a v1 message.
+type legacyMessageWrapper struct{ m protoreflect.ProtoMessage }
+
+func (m legacyMessageWrapper) Reset() { proto.Reset(m.m) }
+func (m legacyMessageWrapper) String() string { return Export{}.MessageStringOf(m.m) }
+func (m legacyMessageWrapper) ProtoMessage() {}
+
+// ProtoMessageV1Of converts either a v1 or v2 message to a v1 message.
+func (Export) ProtoMessageV1Of(m message) piface.MessageV1 {
+ switch mv := m.(type) {
+ case piface.MessageV1:
+ return mv
+ case Unwrapper:
+ return Export{}.ProtoMessageV1Of(mv.ProtoUnwrap())
+ case protoreflect.ProtoMessage:
+ return legacyMessageWrapper{mv}
+ default:
+ panic(fmt.Sprintf("message %T is neither a v1 or v2 Message", m))
}
- return legacyWrapMessage(reflect.ValueOf(m)).ProtoReflect()
}
-// MessageTypeOf returns the protoreflect.MessageType for m.
-func (Export) MessageTypeOf(m message) pref.MessageType {
- if mv, ok := m.(pref.ProtoMessage); ok {
- return mv.ProtoReflect().Type()
+// ProtoMessageV2Of converts either a v1 or v2 message to a v2 message.
+func (Export) ProtoMessageV2Of(m message) pref.ProtoMessage {
+ switch mv := m.(type) {
+ case protoreflect.ProtoMessage:
+ return mv
+ case legacyMessageWrapper:
+ return mv.m
+ case piface.MessageV1:
+ return legacyWrapMessage(reflect.ValueOf(mv))
+ default:
+ panic(fmt.Sprintf("message %T is neither a v1 or v2 Message", m))
}
- return legacyLoadMessageInfo(reflect.TypeOf(m))
+}
+
+// MessageOf returns the protoreflect.Message interface over m.
+func (Export) MessageOf(m message) pref.Message {
+ switch mv := m.(type) {
+ case pref.ProtoMessage:
+ return mv.ProtoReflect()
+ case legacyMessageWrapper:
+ return mv.m.ProtoReflect()
+ case piface.MessageV1:
+ return legacyWrapMessage(reflect.ValueOf(mv)).ProtoReflect()
+ default:
+ panic(fmt.Sprintf("message %T is neither a v1 or v2 Message", m))
+ }
}
// MessageDescriptorOf returns the protoreflect.MessageDescriptor for m.
func (Export) MessageDescriptorOf(m message) pref.MessageDescriptor {
- if mv, ok := m.(pref.ProtoMessage); ok {
+ switch mv := m.(type) {
+ case pref.ProtoMessage:
return mv.ProtoReflect().Descriptor()
+ case legacyMessageWrapper:
+ return mv.m.ProtoReflect().Descriptor()
+ case piface.MessageV1:
+ return LegacyLoadMessageDesc(reflect.TypeOf(mv))
+ default:
+ panic(fmt.Sprintf("message %T is neither a v1 or v2 Message", m))
}
- return LegacyLoadMessageDesc(reflect.TypeOf(m))
+}
+
+// MessageTypeOf returns the protoreflect.MessageType for m.
+func (Export) MessageTypeOf(m message) pref.MessageType {
+ switch mv := m.(type) {
+ case pref.ProtoMessage:
+ return mv.ProtoReflect().Type()
+ case legacyMessageWrapper:
+ return mv.m.ProtoReflect().Type()
+ case piface.MessageV1:
+ return legacyLoadMessageInfo(reflect.TypeOf(mv))
+ default:
+ panic(fmt.Sprintf("message %T is neither a v1 or v2 Message", m))
+ }
}
// MessageStringOf returns the message value as a string,
diff --git a/internal/impl/legacy_test.go b/internal/impl/legacy_test.go
index 3ff3f3c..a231fec 100644
--- a/internal/impl/legacy_test.go
+++ b/internal/impl/legacy_test.go
@@ -554,9 +554,17 @@
Enum int32
)
+func (*MessageA) Reset() { panic("not implemented") }
+func (*MessageA) String() string { panic("not implemented") }
+func (*MessageA) ProtoMessage() { panic("not implemented") }
func (*MessageA) Descriptor() ([]byte, []int) { return concurrentFD, []int{0} }
+
+func (*MessageB) Reset() { panic("not implemented") }
+func (*MessageB) String() string { panic("not implemented") }
+func (*MessageB) ProtoMessage() { panic("not implemented") }
func (*MessageB) Descriptor() ([]byte, []int) { return concurrentFD, []int{1} }
-func (Enum) EnumDescriptor() ([]byte, []int) { return concurrentFD, []int{0} }
+
+func (Enum) EnumDescriptor() ([]byte, []int) { return concurrentFD, []int{0} }
var concurrentFD = func() []byte {
b, _ := proto.Marshal(pdesc.ToFileDescriptorProto(mustMakeFileDesc(`