fix: don't cache array/slice length when traversing.

atombender · atombender · commit f2ba02b77978 · 2022-04-28T11:18:10.000+02:00
diff --git a/pointers.go b/pointers.go
@@ -118,8 +118,7 @@ func (pv *pointerVisitor) consider(v reflect.Value) {
 	// Now descend into any children of this value
 	switch v.Kind() {
 	case reflect.Slice, reflect.Array:
-		numEntries := v.Len()
-		for i := 0; i < numEntries; i++ {
+		for i := 0; i < v.Len(); i++ {
 			pv.consider(v.Index(i))
 		}
 

Original file line number	Diff line number	Diff line change
`@@ -118,8 +118,7 @@ func (pv *pointerVisitor) consider(v reflect.Value) {`
`118`	`118`	`// Now descend into any children of this value`
`119`	`119`	`switch v.Kind() {`
`120`	`120`	`case reflect.Slice, reflect.Array:`
`121`		`- numEntries := v.Len()`
`122`		`- for i := 0; i < numEntries; i++ {`
	`121`	`+ for i := 0; i < v.Len(); i++ {`
`123`	`122`	`pv.consider(v.Index(i))`
`124`	`123`	`}`
`125`	`124`