diff options
| author | Daniel Martí <mvdan@mvdan.cc> | 2018-08-26 08:35:24 -0600 |
|---|---|---|
| committer | Daniel Martí <mvdan@mvdan.cc> | 2018-08-26 17:09:27 +0000 |
| commit | 21af0c1699909454baa4fc6890fe9a1d337faf9c (patch) | |
| tree | fc0da113fd08a13d3dabd7c168486988e4d01740 /src/encoding/json/encode.go | |
| parent | eae5fc88c13d6902a7fe9c595fb02155eb037cbe (diff) | |
| download | go-21af0c1699909454baa4fc6890fe9a1d337faf9c.tar.xz | |
encoding/json: get rid of the stream_test.go TODOs
TestRawMessage now passes without the need for the RawMessage field to
be a pointer. The TODO dates all the way back to 2010, so I presume the
issue has since been fixed.
TestNullRawMessage tested the decoding of a JSON null into a
*RawMessage. The existing behavior was correct, but for the sake of
completeness a non-pointer RawMessage field has been added too. The
non-pointer field behaves differently, as one can read in the docs:
To unmarshal JSON into a value implementing the Unmarshaler
interface, Unmarshal calls that value's UnmarshalJSON method,
including when the input is a JSON null.
Change-Id: Iabaed75d4ed10ea427d135ee1b80c6e6b83b2e6e
Reviewed-on: https://go-review.googlesource.com/131377
Run-TryBot: Daniel Martí <mvdan@mvdan.cc>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Diffstat (limited to 'src/encoding/json/encode.go')
0 files changed, 0 insertions, 0 deletions
