comment multiple dagstore error checking

whyrusleeping · whyrusleeping · commit 02e14c4d3d7e · 2015-11-17T10:17:26.000-08:00
License: MIT
Signed-off-by: Jeromy &lt;jeromyj@gmail.com&gt;
diff --git a/merkledag/utils/utils.go b/merkledag/utils/utils.go
@@ -98,9 +98,12 @@ func (e *Editor) insertNodeAtPath(ctx context.Context, root *dag.Node, path []st
 			nd = create()
 			err = nil // no longer an error case
 		} else if err == dag.ErrNotFound {
+			// try finding it in our source dagstore
 			nd, err = root.GetLinkedNode(ctx, e.src, path[0])
 		}
 
+		// if we receive an ErrNotFound, then our second 'GetLinkedNode' call
+		// also fails, we want to error out
 		if err != nil {
 			return nil, err
 		}
@@ -153,6 +156,7 @@ func (e *Editor) rmLink(ctx context.Context, root *dag.Node, path []string) (*da
 		return root, nil
 	}
 
+	// search for node in both tmp dagstore and source dagstore
 	nd, err := root.GetLinkedNode(ctx, e.tmp, path[0])
 	if err == dag.ErrNotFound {
 		nd, err = root.GetLinkedNode(ctx, e.src, path[0])

Original file line number	Diff line number	Diff line change
`@@ -98,9 +98,12 @@ func (e Editor) insertNodeAtPath(ctx context.Context, root dag.Node, path []st`
`98`	`98`	`nd = create()`
`99`	`99`	`err = nil // no longer an error case`
`100`	`100`	`} else if err == dag.ErrNotFound {`
	`101`	`+ // try finding it in our source dagstore`
`101`	`102`	`nd, err = root.GetLinkedNode(ctx, e.src, path[0])`
`102`	`103`	`}`
`103`	`104`
	`105`	`+ // if we receive an ErrNotFound, then our second 'GetLinkedNode' call`
	`106`	`+ // also fails, we want to error out`
`104`	`107`	`if err != nil {`
`105`	`108`	`return nil, err`
`106`	`109`	`}`
`@@ -153,6 +156,7 @@ func (e Editor) rmLink(ctx context.Context, root dag.Node, path []string) (*da`
`153`	`156`	`return root, nil`
`154`	`157`	`}`
`155`	`158`
	`159`	`+ // search for node in both tmp dagstore and source dagstore`
`156`	`160`	`nd, err := root.GetLinkedNode(ctx, e.tmp, path[0])`
`157`	`161`	`if err == dag.ErrNotFound {`
`158`	`162`	`nd, err = root.GetLinkedNode(ctx, e.src, path[0])`