tree 8d2a0e2ccfd69d373470321918f7f7480bb8aa7c
parent dd380ab64d25a7a7bc4aa94ea1a1d43d396ba4f3
author Damien Neil <dneil@google.com> 1562697649 -0700
committer Damien Neil <dneil@google.com> 1562701762 +0000

proto, internal/impl: make IsInitialized more consistent

Make the fast-path and slow-path versions of IsInitialized report
exactly the same errors: An errors.RequiredNotSet containing the
full name of one of the unset required fields.

Bugfix: Fast-path IsInitialized on a nil message reports an error only
when the message directly contains required fields.

Bugfix: Include fast-path IsInitialized in legacy messageIfaceWrapper.

Fixes golang/protobuf#887

Change-Id: Ia5e4b386f8c23f6f855d995f4a098b1338acbae3
Reviewed-on: https://go-review.googlesource.com/c/protobuf/+/185397
Reviewed-by: Joe Tsai <thebrokentoaster@gmail.com>