mirror of https://github.com/golang/go.git
c700c23cef
The upcoming built-in zero value provides an idiomatic way to test for zero by comparing to the zero literal: v == zero. The reflect package is meant to provide a programmatic way to perform operations that the Go language itself provides. Thus, it seems prudent that reflect.ValueOf(&v).Elem().IsZero() is identical to v == zero. This change alters the behavior of Value.IsZero in two concrete ways: * negative zero is identical to zero * blank fields in a struct are ignored Prior to this change, we were already in an inconsistent state due to a regression introduced by CL 411478. The new behavior was already the case for comparable composite types. This change makes it consistent for all other types (in particular incomparable composite types and standalone numbers). Updates #61372 Fixes #61827 Change-Id: Id23fb97eb3b8921417cc75a1d3ead963e22dc3d9 Reviewed-on: https://go-review.googlesource.com/c/go/+/517777 Reviewed-by: Russ Cox <rsc@golang.org> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Reviewed-by: Ian Lance Taylor <iant@google.com> |
||
|---|---|---|
| .. | ||
| internal | ||
| abi.go | ||
| abi_test.go | ||
| all_test.go | ||
| arena.go | ||
| asm_386.s | ||
| asm_amd64.s | ||
| asm_arm.s | ||
| asm_arm64.s | ||
| asm_loong64.s | ||
| asm_mips64x.s | ||
| asm_mipsx.s | ||
| asm_ppc64x.s | ||
| asm_riscv64.s | ||
| asm_s390x.s | ||
| asm_wasm.s | ||
| benchmark_test.go | ||
| deepequal.go | ||
| example_test.go | ||
| export_test.go | ||
| float32reg_generic.go | ||
| float32reg_ppc64x.s | ||
| float32reg_riscv64.s | ||
| makefunc.go | ||
| nih_test.go | ||
| set_test.go | ||
| stubs_ppc64x.go | ||
| stubs_riscv64.go | ||
| swapper.go | ||
| tostring_test.go | ||
| type.go | ||
| type_test.go | ||
| value.go | ||
| visiblefields.go | ||
| visiblefields_test.go | ||