qbit/go
1
0
Fork 0
mirror of https://github.com/golang/go synced 2024-11-17 20:04:47 -07:00
Code Releases Activity

strings: avoid unnecessary variable setting

Browse Source 
We initialize fieldStart to 0, then set it to i without ever reading
0, so we might as well just initialize it to i.

Change-Id: I17905b25d54a62b6bc76f915353756ed5eb6972b
Reviewed-on: https://go-review.googlesource.com/52933
Reviewed-by: Martin Möhrmann <moehrmann@google.com>
Reviewed-by: Avelino <t@avelino.xxx>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
This commit is contained in:
Kevin Burke 2017-08-02 22:24:22 -07:00 committed by Brad Fitzpatrick
parent 254f8ea9ea
commit e93eb2843c
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/strings/strings.go
View File

@ -363,7 +363,6 @@ func Fields(s string) []string {
// a non-ASCII rune needs to be decoded and checked
// if it corresponds to a space.
a := make([]string, 0, n)
fieldStart := 0
i := 0
// Skip spaces in the front of the input.
for i < len(s) {
@ -380,7 +379,7 @@ func Fields(s string) []string {
i += w
}
}
fieldStart = i
fieldStart := i
for i < len(s) {
if c := s[i]; c < utf8.RuneSelf {
if asciiSpace[c] == 0 {