]> Cypherpunks repositories - gostls13.git/commit
encoding/json: additional tests and fixes for []typedByte encoding/decoding
authorRuss Cox <rsc@golang.org>
Mon, 23 May 2016 16:21:57 +0000 (12:21 -0400)
committerRuss Cox <rsc@golang.org>
Tue, 24 May 2016 13:35:36 +0000 (13:35 +0000)
commit12610236375e2e981e370508548f3d51920df7e2
tree76620b082221af4101b5b148dc34af7c8f685747
parentab4414773e27624abf4361e48a0ca0979e804970
encoding/json: additional tests and fixes for []typedByte encoding/decoding

CL 19725 changed the encoding of []typedByte to look for
typedByte.MarshalJSON and typedByte.MarshalText.
Previously it was handled like []byte, producing a base64 encoding of the underlying byte data.

CL 19725 forgot to look for (*typedByte).MarshalJSON and (*typedByte).MarshalText,
as the marshaling of other slices would. Add test and fix for those.

This CL also adds tests that the decoder can handle both the old and new encodings.
(This was true even in Go 1.6, which is the only reason we can consider this
not an incompatible change.)

For #13783.

Change-Id: I7cab8b6c0154a7f2d09335b7fa23173bcf856c37
Reviewed-on: https://go-review.googlesource.com/23294
Reviewed-by: Ian Lance Taylor <iant@golang.org>
Run-TryBot: Russ Cox <rsc@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Andrew Gerrand <adg@golang.org>
src/encoding/json/decode_test.go
src/encoding/json/encode.go