From: Dmitri Shuralyov Date: Sun, 11 May 2014 01:06:58 +0000 (-0700) Subject: cmd/go: simplify code, reduce allocations. X-Git-Tag: go1.3beta2~103 X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=4118665775dd21b2f244e763bfcccba18902d682;p=gostls13.git cmd/go: simplify code, reduce allocations. This is a trivial change to make use of an existing `nl` byte slice containing a single '\n' character. It's already declared and used in another place in this file, so it might as well be used in the other location instead of a new slice literal. There should be no change in behavior, aside from potentially less allocations. This is my first CL, so I wanted to use a simple, hopefully non-controversial, minor improvement to get more comfortable with golang contribution process. LGTM=bradfitz R=golang-codereviews, bradfitz CC=golang-codereviews https://golang.org/cl/97280043 --- diff --git a/src/cmd/go/list.go b/src/cmd/go/list.go index 63cd4f4f6f..0ead435023 100644 --- a/src/cmd/go/list.go +++ b/src/cmd/go/list.go @@ -161,7 +161,7 @@ func runList(cmd *Command, args []string) { fatalf("%s", err) } if out.NeedNL() { - out.Write([]byte{'\n'}) + out.Write(nl) } } }