diff options
| author | Erik Staab <estaab@google.com> | 2016-09-03 15:57:48 -0700 |
|---|---|---|
| committer | Brad Fitzpatrick <bradfitz@golang.org> | 2016-09-06 00:52:26 +0000 |
| commit | 66121ce8a95f587be8641f79c66c8536660a16d5 (patch) | |
| tree | eeb489a54fb6aad2666dfd1086efccc8d920df0c /src/runtime/mfinal.go | |
| parent | 5514332ed9e770bfb37011f6500a27be53f905c4 (diff) | |
| download | go-66121ce8a95f587be8641f79c66c8536660a16d5.tar.gz go-66121ce8a95f587be8641f79c66c8536660a16d5.zip | |
runtime: remove redundant expression from SetFinalizer
The previous if condition already checks the same expression and doesn't
have side effects.
Change-Id: Ieaf30a786572b608d0a883052b45fd3f04bc6147
Reviewed-on: https://go-review.googlesource.com/28475
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Diffstat (limited to 'src/runtime/mfinal.go')
| -rw-r--r-- | src/runtime/mfinal.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/runtime/mfinal.go b/src/runtime/mfinal.go index 14ebec81bf..9bad6f0aa3 100644 --- a/src/runtime/mfinal.go +++ b/src/runtime/mfinal.go @@ -352,7 +352,7 @@ func SetFinalizer(obj interface{}, finalizer interface{}) { if ft.dotdotdot() { throw("runtime.SetFinalizer: cannot pass " + etyp.string() + " to finalizer " + ftyp.string() + " because dotdotdot") } - if ft.dotdotdot() || ft.inCount != 1 { + if ft.inCount != 1 { throw("runtime.SetFinalizer: cannot pass " + etyp.string() + " to finalizer " + ftyp.string()) } fint := ft.in()[0] |