set data and links nil if not present

whyrusleeping · whyrusleeping · commit dbb9ec4ecc60 · 2015-11-02T23:19:07.000-08:00
License: MIT
Signed-off-by: Jeromy &lt;jeromyj@gmail.com&gt;
diff --git a/merkledag/coding.go b/merkledag/coding.go
@@ -50,7 +50,9 @@ func (n *Node) Marshal() ([]byte, error) {
 
 func (n *Node) getPBNode() *pb.PBNode {
 	pbn := &pb.PBNode{}
-	pbn.Links = make([]*pb.PBLink, len(n.Links))
+	if len(n.Links) > 0 {
+		pbn.Links = make([]*pb.PBLink, len(n.Links))
+	}
 
 	sort.Stable(LinkSlice(n.Links)) // keep links sorted
 	for i, l := range n.Links {
@@ -60,7 +62,9 @@ func (n *Node) getPBNode() *pb.PBNode {
 		pbn.Links[i].Hash = []byte(l.Hash)
 	}
 
-	pbn.Data = n.Data
+	if len(n.Data) > 0 {
+		pbn.Data = n.Data
+	}
 	return pbn
 }
 
diff --git a/merkledag/node.go b/merkledag/node.go
@@ -176,11 +176,15 @@ func (n *Node) GetLinkedNode(ctx context.Context, ds DAGService, name string) (*
 // NOTE: does not make copies of Node objects in the links.
 func (n *Node) Copy() *Node {
 	nnode := new(Node)
-	nnode.Data = make([]byte, len(n.Data))
-	copy(nnode.Data, n.Data)
+	if len(n.Data) > 0 {
+		nnode.Data = make([]byte, len(n.Data))
+		copy(nnode.Data, n.Data)
+	}
 
-	nnode.Links = make([]*Link, len(n.Links))
-	copy(nnode.Links, n.Links)
+	if len(n.Links) > 0 {
+		nnode.Links = make([]*Link, len(n.Links))
+		copy(nnode.Links, n.Links)
+	}
 	return nnode
 }
 
diff --git a/test/sharness/t0051-object.sh b/test/sharness/t0051-object.sh
@@ -234,6 +234,12 @@ test_object_cmd() {
 
 	test_patch_create_path $EMPTY a/b/b/b/b $FILE
 
+	test_expect_success "can create blank object" '
+		BLANK=$(ipfs object new)
+	'
+
+	test_patch_create_path $BLANK a $FILE
+
 	test_expect_success "create bad path fails" '
 		test_must_fail ipfs object patch --create $EMPTY add-link / $FILE
 	'
