util/deephash: remove unused stack slice in typeIsRecursive (#5363)

No operation ever reads from this variable.

Signed-off-by: Joe Tsai <joetsai@digital-static.net>
pull/5385/head
Joe Tsai 2 years ago committed by GitHub
parent c200229f9e
commit 23ec3c104a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

@ -646,8 +646,6 @@ func getTypeInfoLocked(t reflect.Type, incomplete map[reflect.Type]*typeInfo) *t
func typeIsRecursive(t reflect.Type) bool { func typeIsRecursive(t reflect.Type) bool {
inStack := map[reflect.Type]bool{} inStack := map[reflect.Type]bool{}
var stack []reflect.Type
var visitType func(t reflect.Type) (isRecursiveSoFar bool) var visitType func(t reflect.Type) (isRecursiveSoFar bool)
visitType = func(t reflect.Type) (isRecursiveSoFar bool) { visitType = func(t reflect.Type) (isRecursiveSoFar bool) {
switch t.Kind() { switch t.Kind() {
@ -678,11 +676,9 @@ func typeIsRecursive(t reflect.Type) bool {
if inStack[t] { if inStack[t] {
return true return true
} }
stack = append(stack, t)
inStack[t] = true inStack[t] = true
defer func() { defer func() {
delete(inStack, t) delete(inStack, t)
stack = stack[:len(stack)-1]
}() }()
switch t.Kind() { switch t.Kind() {

Loading…
Cancel
Save