From c925c4084620fe4701b202e3fa813474abf79af4 Mon Sep 17 00:00:00 2001 From: Tobie Morgan Hitchcock Date: Mon, 27 Nov 2017 18:32:47 +0000 Subject: [PATCH] Yes we do need to copy all maps and slices --- db/fetch.go | 2 -- 1 file changed, 2 deletions(-) diff --git a/db/fetch.go b/db/fetch.go index f10ca165..51278f4e 100644 --- a/db/fetch.go +++ b/db/fetch.go @@ -43,10 +43,8 @@ func (e *executor) fetch(ctx context.Context, val interface{}, doc *data.Doc) (o case []byte: return string(val), nil case []interface{}: - // TODO do we really need to copy? return deep.Copy(val), nil case map[string]interface{}: - // TODO do we really need to copy? return deep.Copy(val), nil case *sql.Ident: