From 47f4ae1a78cb09999f57b34a38913047d5d99a53 Mon Sep 17 00:00:00 2001 From: Evan Shaw Date: Tue, 29 Mar 2011 01:27:38 -0400 Subject: [PATCH] bytes, strings: simplify Join R=gri, rsc CC=golang-dev https://golang.org/cl/4300044 --- src/pkg/bytes/bytes.go | 18 ++++-------------- src/pkg/strings/strings.go | 18 ++++-------------- 2 files changed, 8 insertions(+), 28 deletions(-) diff --git a/src/pkg/bytes/bytes.go b/src/pkg/bytes/bytes.go index bfe2ef39db..c12a135738 100644 --- a/src/pkg/bytes/bytes.go +++ b/src/pkg/bytes/bytes.go @@ -293,20 +293,10 @@ func Join(a [][]byte, sep []byte) []byte { } b := make([]byte, n) - bp := 0 - for i := 0; i < len(a); i++ { - s := a[i] - for j := 0; j < len(s); j++ { - b[bp] = s[j] - bp++ - } - if i+1 < len(a) { - s = sep - for j := 0; j < len(s); j++ { - b[bp] = s[j] - bp++ - } - } + bp := copy(b, a[0]) + for _, s := range a[1:] { + bp += copy(b[bp:], sep) + bp += copy(b[bp:], s) } return b } diff --git a/src/pkg/strings/strings.go b/src/pkg/strings/strings.go index 44dcf99b65..93c7c46473 100644 --- a/src/pkg/strings/strings.go +++ b/src/pkg/strings/strings.go @@ -275,20 +275,10 @@ func Join(a []string, sep string) string { } b := make([]byte, n) - bp := 0 - for i := 0; i < len(a); i++ { - s := a[i] - for j := 0; j < len(s); j++ { - b[bp] = s[j] - bp++ - } - if i+1 < len(a) { - s = sep - for j := 0; j < len(s); j++ { - b[bp] = s[j] - bp++ - } - } + bp := copy(b, a[0]) + for _, s := range a[1:] { + bp += copy(b[bp:], sep) + bp += copy(b[bp:], s) } return string(b) }