]> Cypherpunks repositories - gostls13.git/commit
encoding/json: remove suggestion on Unmarshaler with JSON null
authorJoe Tsai <joetsai@digital-static.net>
Sat, 28 Dec 2024 01:24:30 +0000 (17:24 -0800)
committerJoseph Tsai <joetsai@digital-static.net>
Mon, 30 Dec 2024 20:21:48 +0000 (12:21 -0800)
commit15f232456a8741c84ee0bd38dce28978eab6a491
tree91a371f5692c5acc1b6b09ec845b9c6adb3f9c7b
parentba1deb1ceef956bdb3ca5a9570f132cf19ccc9f6
encoding/json: remove suggestion on Unmarshaler with JSON null

It is not true that Unmarshal always treats a JSON null
as being equivalent to a no-op.
For bools, ints, uints, floats, strings, arrays, and structs,
it treats a JSON null as a no-op. However, for []byte, slice,
map, pointer, or interface, it zeros the underlying value.

Remove this suggestion as the actual behavior is inconsistent.

Note that the proposed behavior in v2 Unmarshal is to consistently
zero out the underlying value.

Change-Id: I02cef0bf7919f25cfd0aceb04486d37498761181
Reviewed-on: https://go-review.googlesource.com/c/go/+/638416
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Ian Lance Taylor <iant@google.com>
Run-TryBot: Joseph Tsai <joetsai@digital-static.net>
src/encoding/json/decode.go