From 09004e498919aa5cb2092d8103f4a8b7c4fa3846 Mon Sep 17 00:00:00 2001
From: Jeromy <jeromyj@gmail.com>
Date: Thu, 30 Oct 2014 20:50:24 +0000
Subject: [PATCH] finish addressing PR concerns

---
 unixfs/io/dagreader.go | 5 ++---
 util/testutil/gen.go   | 8 +++++---
 2 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/unixfs/io/dagreader.go b/unixfs/io/dagreader.go
index 307f1d305..804e03438 100644
--- a/unixfs/io/dagreader.go
+++ b/unixfs/io/dagreader.go
@@ -35,12 +35,11 @@ func NewDagReader(n *mdag.Node, serv mdag.DAGService) (io.Reader, error) {
 		// Dont allow reading directories
 		return nil, ErrIsDir
 	case ftpb.Data_File:
-		dr := &DagReader{
+		return &DagReader{
 			node: n,
 			serv: serv,
 			buf:  bytes.NewBuffer(pb.GetData()),
-		}
-		return dr, nil
+		}, nil
 	case ftpb.Data_Raw:
 		// Raw block will just be a single level, return a byte buffer
 		return bytes.NewBuffer(pb.GetData()), nil
diff --git a/util/testutil/gen.go b/util/testutil/gen.go
index 18b88801d..111cbe728 100644
--- a/util/testutil/gen.go
+++ b/util/testutil/gen.go
@@ -9,6 +9,7 @@ import (
 	ds "github.com/jbenet/go-ipfs/Godeps/_workspace/src/github.com/jbenet/go-datastore"
 	bsrv "github.com/jbenet/go-ipfs/blockservice"
 	dag "github.com/jbenet/go-ipfs/merkledag"
+	u "github.com/jbenet/go-ipfs/util"
 )
 
 func GetDAGServ(t testing.TB) dag.DAGService {
@@ -21,7 +22,8 @@ func GetDAGServ(t testing.TB) dag.DAGService {
 }
 
 func RandPeer() peer.Peer {
-	id := make(peer.ID, 16)
+	id := make([]byte, 16)
 	crand.Read(id)
-	return peer.WithID(id)
-}
\ No newline at end of file
+	mhid := u.Hash(id)
+	return peer.WithID(peer.ID(mhid))
+}