tree 0f580c83fae8f7910af7cd60eaffaa7d66a597aa
parent c93f494f4a06d8a4acb050299cd83f058e622fac
author Joe Tsai <joetsai@digital-static.net> 1552337114 -0700
committer Joe Tsai <thebrokentoaster@gmail.com> 1552339411 +0000

internal/impl: split messageWrapper into different types

The previous implementation of messageWrapper implemented both
proto.Message and protoreflect.Message. This made it possible for users
to accidentally rely on this fact when it was actually an internal
implementation detail.

To avoid this, we split the wrapper type into two, ensuring that each
only implements one or the other. Doing so also revealed bugs in our
own code where we accidentally relied on this fact.

Change-Id: I0ff521b5c806b7dcb0b86942bd97e8319d8e8657
Reviewed-on: https://go-review.googlesource.com/c/protobuf/+/166938
Reviewed-by: Damien Neil <dneil@google.com>