diff options
| author | Rob Pike <r@golang.org> | 2022-03-14 13:21:06 +1100 |
|---|---|---|
| committer | Ian Lance Taylor <iant@golang.org> | 2022-04-04 17:28:30 +0000 |
| commit | c58f1bb65f2187d79a5842bb19f4db4cafd22794 (patch) | |
| tree | fb395d191315738c8dfd0fa2cabd0439336906f1 /src/make.bash | |
| parent | ac313524fe4997b80a4221647f0da79d0e07b88e (diff) | |
| download | go-c58f1bb65f2187d79a5842bb19f4db4cafd22794.tar.xz | |
text/template: permit eq and ne funcs to check against nil
The existing code errors out immediately if the argument is not
"comparable", making it impossible to test a slice, map, and so
on from being compared to nil.
Fix by delaying the "comparable" error check until we encounter
an actual check between two non-comparable, non-nil values.
Note for the future: reflect makes it unnecessarily clumsy
to deal with nil values in cases like this. For instance, it
should be possible to check if a value is nil without stepping
around a panic. See the new functions isNil and canCompare
for my (too expensive) workaround.
Fixes #51642
Change-Id: Ic4072698c4910130ea7e3d76e7a148d8a8b88162
Reviewed-on: https://go-review.googlesource.com/c/go/+/392274
Reviewed-by: Ian Lance Taylor <iant@golang.org>
Run-TryBot: Ian Lance Taylor <iant@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Trust: Cherry Mui <cherryyz@google.com>
Diffstat (limited to 'src/make.bash')
0 files changed, 0 insertions, 0 deletions
