diff options
| author | Daniel Martí <mvdan@mvdan.cc> | 2017-09-11 21:51:38 +0200 |
|---|---|---|
| committer | Daniel Martí <mvdan@mvdan.cc> | 2017-09-12 08:58:28 +0000 |
| commit | eb2dc3d3d076f6924b4d25f89de267dc93066ea5 (patch) | |
| tree | 042081fac5d6d633c1ac1fd3f3416734fffccc3c /src/path | |
| parent | 137e4a6c63e8688cba34df7dad81b45f0aaa500c (diff) | |
| download | go-eb2dc3d3d076f6924b4d25f89de267dc93066ea5.tar.xz | |
all: remove strings.Contains check around Replace
It doesn't change the outcome. It might have been useful at some point
to avoid Replace from doing work or allocating. However, nowadays the
func returns early without doing any work if Count returns 0.
Change-Id: Id69dc74042a6e39672b405016484db8b50f43d58
Reviewed-on: https://go-review.googlesource.com/62991
Run-TryBot: Daniel Martí <mvdan@mvdan.cc>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Dave Cheney <dave@cheney.net>
Diffstat (limited to 'src/path')
| -rw-r--r-- | src/path/filepath/path_windows.go | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/path/filepath/path_windows.go b/src/path/filepath/path_windows.go index 0d8b62015c..03542559f8 100644 --- a/src/path/filepath/path_windows.go +++ b/src/path/filepath/path_windows.go @@ -100,9 +100,7 @@ func splitList(path string) []string { // Remove quotes. for i, s := range list { - if strings.Contains(s, `"`) { - list[i] = strings.Replace(s, `"`, ``, -1) - } + list[i] = strings.Replace(s, `"`, ``, -1) } return list |