reflect: switch MessageType.New to return Message
Most usages of New actually prefer to interact with the reflective view
rather than the native Go type. Thus, change New to return that instead.
This parallels reflect.New, which returns the reflective view
(i.e., reflect.Value) instead of native type (i.e., interface{}).
We make the equivalent change to KnownFields.NewMessage, List.NewMessage,
and Map.NewMessage for consistency.
Since this is a subtle change where the type system will not always
catch the changed type, this change was made by both changing the type
and renaming the function to NewXXX and manually looking at every usage
of the the function to ensure that the usage correctly operates
on either the native Go type or the reflective view of the type.
After the entire codebase was cleaned up, a rename was performed to convert
NewXXX back to New.
Change-Id: I153fef627b4bf0a427e4039ce0aaec52e20c7950
Reviewed-on: https://go-review.googlesource.com/c/157077
Reviewed-by: Damien Neil <dneil@google.com>
diff --git a/internal/testprotos/test/test.pb.go b/internal/testprotos/test/test.pb.go
index bb71643..9fb1322 100644
--- a/internal/testprotos/test/test.pb.go
+++ b/internal/testprotos/test/test.pb.go
@@ -1636,44 +1636,44 @@
var xxx_Test_ProtoFile_MessageTypes = [24]protoimpl.MessageType{
{Type: prototype.GoMessage(
xxx_Test_ProtoFile_MessageDescs[0].Reference(),
- func(protoreflect.MessageType) protoreflect.ProtoMessage {
- return new(TestAllTypes)
+ func(protoreflect.MessageType) protoreflect.Message {
+ return xxx_TestAllTypes{new(TestAllTypes)}
},
)},
{Type: prototype.GoMessage(
xxx_Test_ProtoFile_MessageDescs[1].Reference(),
- func(protoreflect.MessageType) protoreflect.ProtoMessage {
- return new(TestAllExtensions)
+ func(protoreflect.MessageType) protoreflect.Message {
+ return xxx_TestAllExtensions{new(TestAllExtensions)}
},
)},
{Type: prototype.GoMessage(
xxx_Test_ProtoFile_MessageDescs[2].Reference(),
- func(protoreflect.MessageType) protoreflect.ProtoMessage {
- return new(OptionalGroupExtension)
+ func(protoreflect.MessageType) protoreflect.Message {
+ return xxx_OptionalGroupExtension{new(OptionalGroupExtension)}
},
)},
{Type: prototype.GoMessage(
xxx_Test_ProtoFile_MessageDescs[3].Reference(),
- func(protoreflect.MessageType) protoreflect.ProtoMessage {
- return new(RepeatedGroupExtension)
+ func(protoreflect.MessageType) protoreflect.Message {
+ return xxx_RepeatedGroupExtension{new(RepeatedGroupExtension)}
},
)},
{Type: prototype.GoMessage(
xxx_Test_ProtoFile_MessageDescs[4].Reference(),
- func(protoreflect.MessageType) protoreflect.ProtoMessage {
- return new(TestAllTypes_NestedMessage)
+ func(protoreflect.MessageType) protoreflect.Message {
+ return xxx_TestAllTypes_NestedMessage{new(TestAllTypes_NestedMessage)}
},
)},
{Type: prototype.GoMessage(
xxx_Test_ProtoFile_MessageDescs[5].Reference(),
- func(protoreflect.MessageType) protoreflect.ProtoMessage {
- return new(TestAllTypes_OptionalGroup)
+ func(protoreflect.MessageType) protoreflect.Message {
+ return xxx_TestAllTypes_OptionalGroup{new(TestAllTypes_OptionalGroup)}
},
)},
{Type: prototype.GoMessage(
xxx_Test_ProtoFile_MessageDescs[6].Reference(),
- func(protoreflect.MessageType) protoreflect.ProtoMessage {
- return new(TestAllTypes_RepeatedGroup)
+ func(protoreflect.MessageType) protoreflect.Message {
+ return xxx_TestAllTypes_RepeatedGroup{new(TestAllTypes_RepeatedGroup)}
},
)},
{ /* no message type for TestAllTypes_MapInt32Int32Entry */ },