only visit nodes in EnumerateChildrenAsync when asked

No idea why this was changed this was introduced in:

08f342e (part of ipfs#3598)

License: MIT
Signed-off-by: Steven Allen <[email protected]>

Author: Stebalien

merkledag.go

@@ -319,17 +319,17 @@ func EnumerateChildrenAsync(ctx context.Context, getLinks GetLinks, c *cid.Cid,
 	for i := 0; i < FetchGraphConcurrency; i++ {
 		go func() {
 			for ic := range feed {
-				links, err := getLinks(ctx, ic)
-				if err != nil {
-					errChan <- err
-					return
-				}
-
 				setlk.Lock()
-				unseen := visit(ic)
+				shouldVisit := visit(ic)
 				setlk.Unlock()
 
-				if unseen {
+				if shouldVisit {
+					links, err := getLinks(ctx, ic)
+					if err != nil {
+						errChan <- err
+						return
+					}
+
 					select {
 					case out <- links:
 					case <-fetchersCtx.Done():