tree f65f6dc4d2970e1de24a152bf4f8f3364a36f280
parent ac503595923892665044c980874a932c8a51d946
author Herbie Ong <herbie@google.com> 1554331361 -0700
committer Herbie Ong <herbie@google.com> 1554336234 +0000

encoding/textpb: clean-up code and fix error handling inside closures

In marshalMap, replace Map.Range and sortMap with
internal/mapsort.Range. Also, make sure to capture fatal error properly
for return.

In appendExtensions, capture fatal error properly for return. Added a
testcase that shows this was a bug.

In unmarshalAny, remove unnecessary checks and use internal/fieldnum for
Any's field numbers.

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