bytes: change the Copy return type to non pointer

Non pointer slice is less confusing than with pointer.

1 files changed, 4 insertions, 5 deletions

diff --git a/lib/bytes/bytes.go b/lib/bytes/bytes.go
index 5c9da0be..94208a5e 100644
--- a/lib/bytes/bytes.go
+++ b/lib/bytes/bytes.go
@@ -101,13 +101,12 @@ func Concat(sb []byte, args ...interface{}) (out []byte) {
 //
 // Copy slice of bytes from parameter.
 //
-func Copy(src []byte) (dst *[]byte) {
+func Copy(src []byte) (dst []byte) {
 	if len(src) == 0 {
-		return nil
+		return
 	}
-	cp := make([]byte, len(src))
-	copy(cp, src)
-	dst = &cp
+	dst = make([]byte, len(src))
+	copy(dst, src)
 	return
 }