noth/surrealpatch
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Yes we do need to copy all maps and slices

Browse source
This commit is contained in:
Tobie Morgan Hitchcock 2017-11-27 18:32:47 +00:00
parent 287951a37f
commit c925c40846
1 changed files with 0 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
db/fetch.go
View file

@ -43,10 +43,8 @@ func (e *executor) fetch(ctx context.Context, val interface{}, doc *data.Doc) (o
case []byte: case []byte:
return string(val), nil return string(val), nil
case []interface{}: case []interface{}:
// TODO do we really need to copy?
return deep.Copy(val), nil return deep.Copy(val), nil
case map[string]interface{}: case map[string]interface{}:
// TODO do we really need to copy?
return deep.Copy(val), nil return deep.Copy(val), nil
case *sql.Ident: case *sql.Ident: