From: Marko Kungla Date: Sat, 7 Sep 2019 09:07:08 +0000 (+0300) Subject: reflect: enhance docs for IsZero and IsValid X-Git-Tag: go1.14beta1~1143 X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=a5026af57c7934f0856cfd4b539a7859d85a0474;p=gostls13.git reflect: enhance docs for IsZero and IsValid Make it clear that IsValid checks that we have valid reflect.Value and not the value of `v` fixes #34152 Change-Id: Ib3d359eeb3a82bf733b9ed17c777fc4c143bc29c Reviewed-on: https://go-review.googlesource.com/c/go/+/193841 Reviewed-by: Ian Lance Taylor --- diff --git a/src/internal/reflectlite/value.go b/src/internal/reflectlite/value.go index 308cf98fc8..6a493938f5 100644 --- a/src/internal/reflectlite/value.go +++ b/src/internal/reflectlite/value.go @@ -305,7 +305,7 @@ func (v Value) IsNil() bool { // IsValid reports whether v represents a value. // It returns false if v is the zero Value. // If IsValid returns false, all other methods except String panic. -// Most functions and methods never return an invalid value. +// Most functions and methods never return an invalid Value. // If one does, its documentation states the conditions explicitly. func (v Value) IsValid() bool { return v.flag != 0 diff --git a/src/reflect/value.go b/src/reflect/value.go index 2e80bfe77f..7fec09962c 100644 --- a/src/reflect/value.go +++ b/src/reflect/value.go @@ -1076,7 +1076,7 @@ func (v Value) IsNil() bool { // IsValid reports whether v represents a value. // It returns false if v is the zero Value. // If IsValid returns false, all other methods except String panic. -// Most functions and methods never return an invalid value. +// Most functions and methods never return an invalid Value. // If one does, its documentation states the conditions explicitly. func (v Value) IsValid() bool { return v.flag != 0