]> Cypherpunks repositories - gostls13.git/commitdiff
all: revert "all: prefer strings.LastIndexByte over strings.LastIndex"
authorMarvin Stenger <marvin.stenger94@gmail.com>
Thu, 5 Oct 2017 13:50:11 +0000 (15:50 +0200)
committerIan Lance Taylor <iant@golang.org>
Thu, 5 Oct 2017 23:19:42 +0000 (23:19 +0000)
This reverts https://golang.org/cl/66372.

Updates #22148

Change-Id: I3e94af3dfc11a2883bf28e1d5e1f32f98760b3ee
Reviewed-on: https://go-review.googlesource.com/68431
Reviewed-by: Ian Lance Taylor <iant@golang.org>
25 files changed:
src/archive/tar/writer.go
src/cmd/doc/main.go
src/cmd/fix/fix.go
src/cmd/vet/print.go
src/crypto/tls/handshake_client.go
src/crypto/tls/tls.go
src/debug/gosym/symtab.go
src/encoding/xml/marshal.go
src/go/build/build.go
src/go/doc/testdata/testing.go
src/go/printer/printer.go
src/go/types/resolver.go
src/internal/trace/parser.go
src/net/http/cookiejar/jar.go
src/net/http/cookiejar/jar_test.go
src/net/http/http.go
src/net/http/request.go
src/net/http/transport.go
src/net/mail/message.go
src/net/rpc/server.go
src/net/url/url.go
src/os/exec/lp_windows.go
src/path/path.go
src/testing/testing.go
src/time/mono_test.go

index 0afb5577eba1e2acc566af7afc0f6a935fd4ae47..1e5b76f58f5cdddb38549f9442a0e97bacc392dc 100644 (file)
@@ -376,7 +376,7 @@ func splitUSTARPath(name string) (prefix, suffix string, ok bool) {
                length--
        }
 
-       i := strings.LastIndexByte(name[:length], '/')
+       i := strings.LastIndex(name[:length], "/")
        nlen := len(name) - i - 1 // nlen is length of suffix
        plen := i                 // plen is length of prefix
        if i <= 0 || nlen > nameSize || nlen == 0 || plen > prefixSize {
index e0a71121933a3c97c234ee0a5676723770b13e8c..de275403a2189b8c05638637c8b3c29d34818695 100644 (file)
@@ -205,7 +205,7 @@ func parseArgs(args []string) (pkg *build.Package, path, symbol string, more boo
        }
        // If it has a slash, it must be a package path but there is a symbol.
        // It's the last package path we care about.
-       slash := strings.LastIndexByte(arg, '/')
+       slash := strings.LastIndex(arg, "/")
        // There may be periods in the package path before or after the slash
        // and between a symbol and method.
        // Split the string at various periods to see what we find.
index ebed82fbeb604ddb0ad3f7fd4a8d4927a03e0c13..03c828a5816499590a295bd3ef98cb7b72555708 100644 (file)
@@ -719,7 +719,7 @@ func usesImport(f *ast.File, path string) (used bool) {
        case "<nil>":
                // If the package name is not explicitly specified,
                // make an educated guess. This is not guaranteed to be correct.
-               lastSlash := strings.LastIndexByte(path, '/')
+               lastSlash := strings.LastIndex(path, "/")
                if lastSlash == -1 {
                        name = path
                } else {
index 453a072565fa2154dbd23414c448a02dafbdf732..21bb0d09972a698f743b020ae4690f8d8001bea7 100644 (file)
@@ -38,7 +38,7 @@ func initPrintFlags() {
 
                // Backwards compatibility: skip optional first argument
                // index after the colon.
-               if colon := strings.LastIndexByte(name, ':'); colon > 0 {
+               if colon := strings.LastIndex(name, ":"); colon > 0 {
                        name = name[:colon]
                }
 
index 38d01bf06815d8f05eb1f17634bfe6d0e7c9697d..f8db66279f79b731929acce50051144af89bcebd 100644 (file)
@@ -848,7 +848,7 @@ func hostnameInSNI(name string) string {
        if len(host) > 0 && host[0] == '[' && host[len(host)-1] == ']' {
                host = host[1 : len(host)-1]
        }
-       if i := strings.LastIndexByte(host, '%'); i > 0 {
+       if i := strings.LastIndex(host, "%"); i > 0 {
                host = host[:i]
        }
        if net.ParseIP(host) != nil {
index 1c91daeb1ce2bc2d6d8481b499acfb7aaf6e8ff7..615d1e5576fdaab8c772eadae62a3d146757cc6d 100644 (file)
@@ -122,7 +122,7 @@ func DialWithDialer(dialer *net.Dialer, network, addr string, config *Config) (*
                return nil, err
        }
 
-       colonPos := strings.LastIndexByte(addr, ':')
+       colonPos := strings.LastIndex(addr, ":")
        if colonPos == -1 {
                colonPos = len(addr)
        }
index fd8703b36bc8c449b50de660242f77e5021aa4db..f5f996309509167bb8c40380c1510058eddc88c3 100644 (file)
@@ -40,7 +40,7 @@ func (s *Sym) Static() bool { return s.Type >= 'a' }
 // PackageName returns the package part of the symbol name,
 // or the empty string if there is none.
 func (s *Sym) PackageName() string {
-       pathend := strings.LastIndexByte(s.Name, '/')
+       pathend := strings.LastIndex(s.Name, "/")
        if pathend < 0 {
                pathend = 0
        }
@@ -54,12 +54,12 @@ func (s *Sym) PackageName() string {
 // ReceiverName returns the receiver type name of this symbol,
 // or the empty string if there is none.
 func (s *Sym) ReceiverName() string {
-       pathend := strings.LastIndexByte(s.Name, '/')
+       pathend := strings.LastIndex(s.Name, "/")
        if pathend < 0 {
                pathend = 0
        }
        l := strings.Index(s.Name[pathend:], ".")
-       r := strings.LastIndexByte(s.Name[pathend:], '.')
+       r := strings.LastIndex(s.Name[pathend:], ".")
        if l == -1 || r == -1 || l == r {
                return ""
        }
@@ -68,7 +68,7 @@ func (s *Sym) ReceiverName() string {
 
 // BaseName returns the symbol name without the package or receiver name.
 func (s *Sym) BaseName() string {
-       if i := strings.LastIndexByte(s.Name, '.'); i != -1 {
+       if i := strings.LastIndex(s.Name, "."); i != -1 {
                return s.Name[i+1:]
        }
        return s.Name
index 773f0f2862ee6eb2420e7ee2e4eb0357c8409ffb..42133a75abc056bd1f1dc8fd72e26d5f8f9c8ded 100644 (file)
@@ -333,7 +333,7 @@ func (p *printer) createAttrPrefix(url string) string {
        // Pick a name. We try to use the final element of the path
        // but fall back to _.
        prefix := strings.TrimRight(url, "/")
-       if i := strings.LastIndexByte(prefix, '/'); i >= 0 {
+       if i := strings.LastIndex(prefix, "/"); i >= 0 {
                prefix = prefix[i+1:]
        }
        if prefix == "" || !isName([]byte(prefix)) || strings.Contains(prefix, ":") {
index 8e92b2f5d121d8a20bb0e3e2f70f24caf20ec637..d8163d01723cfa77005b6f356cb7f4051071b726 100644 (file)
@@ -465,7 +465,7 @@ func (e *MultiplePackageError) Error() string {
 }
 
 func nameExt(name string) string {
-       i := strings.LastIndexByte(name, '.')
+       i := strings.LastIndex(name, ".")
        if i < 0 {
                return ""
        }
@@ -610,7 +610,7 @@ func (ctxt *Context) Import(path string, srcDir string, mode ImportMode) (*Packa
                                                }
                                                tried.vendor = append(tried.vendor, dir)
                                        }
-                                       i := strings.LastIndexByte(sub, '/')
+                                       i := strings.LastIndex(sub, "/")
                                        if i < 0 {
                                                break
                                        }
@@ -1064,7 +1064,7 @@ func (ctxt *Context) matchFile(dir, name string, allTags map[string]bool, binary
                return
        }
 
-       i := strings.LastIndexByte(name, '.')
+       i := strings.LastIndex(name, ".")
        if i < 0 {
                i = len(name)
        }
index 066e4a43db7b3fe4a564dc7d35822c52805c68d8..52810f7a564c17389dd6e5f30404dc517ba8d967 100644 (file)
@@ -93,7 +93,7 @@ func decorate(s string, addFileLine bool) string {
                _, file, line, ok := runtime.Caller(3) // decorate + log + public function.
                if ok {
                        // Truncate file name at last file name separator.
-                       if index := strings.LastIndexByte(file, '/'); index >= 0 {
+                       if index := strings.LastIndex(file, "/"); index >= 0 {
                                file = file[index+1:]
                        } else if index = strings.LastIndex(file, "\\"); index >= 0 {
                                file = file[index+1:]
index 2b8dc19fba9d2f4c7467f294248c766592f49612..dbb4bbd90cf205cfbc9304b23ca63ccb81803f71 100644 (file)
@@ -624,7 +624,7 @@ func (p *printer) writeComment(comment *ast.Comment) {
        if strings.HasPrefix(text, linePrefix) && (!pos.IsValid() || pos.Column == 1) {
                // possibly a line directive
                ldir := strings.TrimSpace(text[len(linePrefix):])
-               if i := strings.LastIndexByte(ldir, ':'); i >= 0 {
+               if i := strings.LastIndex(ldir, ":"); i >= 0 {
                        if line, err := strconv.Atoi(ldir[i+1:]); err == nil && line > 0 {
                                // The line directive we are about to print changed
                                // the Filename and Line number used for subsequent
index 253f8a0334ea4f37dec772d664ffef45313fc58f..ba75a0dc2398c67454407af4bb49a936ac33adb0 100644 (file)
@@ -172,7 +172,7 @@ func (check *Checker) importPackage(pos token.Pos, path, dir string) *Package {
                                if i := len(name); i > 0 && name[i-1] == '/' {
                                        name = name[:i-1]
                                }
-                               if i := strings.LastIndexByte(name, '/'); i >= 0 {
+                               if i := strings.LastIndex(name, "/"); i >= 0 {
                                        name = name[i+1:]
                                }
                                imp = NewPackage(path, name)
@@ -516,7 +516,7 @@ func (check *Checker) unusedImports() {
 
 // pkgName returns the package name (last element) of an import path.
 func pkgName(path string) string {
-       if i := strings.LastIndexByte(path, '/'); i >= 0 {
+       if i := strings.LastIndex(path, "/"); i >= 0 {
                path = path[i+1:]
        }
        return path
index 1033d5222b6c9352252f8ecb8aacba6154e5b2d9..a774bf14c96ae77c1e8719dfdb99192a42abf616 100644 (file)
@@ -812,7 +812,7 @@ func symbolize(events []*Event, bin string) error {
                f := &Frame{PC: pc}
                f.Fn = fn[:len(fn)-1]
                f.File = file[:len(file)-1]
-               if colon := strings.LastIndexByte(f.File, ':'); colon != -1 {
+               if colon := strings.LastIndex(f.File, ":"); colon != -1 {
                        ln, err := strconv.Atoi(f.File[colon+1:])
                        if err == nil {
                                f.File = f.File[:colon]
index f147eceb189cd67cc03cf19455d7a039267c2fe1..ef8c35bf0a167700e5fc41f2e66df2978923783f 100644 (file)
@@ -330,7 +330,7 @@ func jarKey(host string, psl PublicSuffixList) string {
 
        var i int
        if psl == nil {
-               i = strings.LastIndexByte(host, '.')
+               i = strings.LastIndex(host, ".")
                if i <= 0 {
                        return host
                }
@@ -349,7 +349,7 @@ func jarKey(host string, psl PublicSuffixList) string {
                // here on, so it is okay if psl.PublicSuffix("www.buggy.psl")
                // returns "com" as the jar key is generated from host.
        }
-       prevDot := strings.LastIndexByte(host[:i-1], '.')
+       prevDot := strings.LastIndex(host[:i-1], ".")
        return host[prevDot+1:]
 }
 
@@ -365,7 +365,7 @@ func defaultPath(path string) string {
                return "/" // Path is empty or malformed.
        }
 
-       i := strings.LastIndexByte(path, '/') // Path starts with "/", so i != -1.
+       i := strings.LastIndex(path, "/") // Path starts with "/", so i != -1.
        if i == 0 {
                return "/" // Path has the form "/abc".
        }
index ed4baae4c616c80171afe90f9e40c5e517fc7279..47fb1abdaafa4a24dd1335b9ce242ba3cf63dc64 100644 (file)
@@ -37,7 +37,7 @@ func (testPSL) PublicSuffix(d string) string {
        if d == "www2.buggy.psl" {
                return "com"
        }
-       return d[strings.LastIndexByte(d, '.')+1:]
+       return d[strings.LastIndex(d, ".")+1:]
 }
 
 // newTestJar creates an empty Jar with testPSL as the public suffix list.
index 7d5b74092e68408d5dbca00a2c1efb17560de776..b95ca89f4094c5f04445c83ad5c3d9069cc777f9 100644 (file)
@@ -35,7 +35,7 @@ func (k *contextKey) String() string { return "net/http context value " + k.name
 
 // Given a string of the form "host", "host:port", or "[ipv6::address]:port",
 // return true if the string includes a port.
-func hasPort(s string) bool { return strings.LastIndexByte(s, ':') > strings.LastIndexByte(s, ']') }
+func hasPort(s string) bool { return strings.LastIndex(s, ":") > strings.LastIndex(s, "]") }
 
 // removeEmptyPort strips the empty port in ":port" to ""
 // as mandated by RFC 3986 Section 6.2.3.
index 801d00ef2f03a476f93a2eac20eaf92ab77ebc1a..870af85e04a002b1fd078e71441a4d1671641c2e 100644 (file)
@@ -688,11 +688,11 @@ func removeZone(host string) string {
        if !strings.HasPrefix(host, "[") {
                return host
        }
-       i := strings.LastIndexByte(host, ']')
+       i := strings.LastIndex(host, "]")
        if i < 0 {
                return host
        }
-       j := strings.LastIndexByte(host[:i], '%')
+       j := strings.LastIndex(host[:i], "%")
        if j < 0 {
                return host
        }
index 034d016cb36f9be8b00247075c71e3d8b36b13dd..5f2ace7b4b4fcf703f00c10a987720f35f0592ea 100644 (file)
@@ -1236,7 +1236,7 @@ func useProxy(addr string) bool {
 
        addr = strings.ToLower(strings.TrimSpace(addr))
        if hasPort(addr) {
-               addr = addr[:strings.LastIndexByte(addr, ':')]
+               addr = addr[:strings.LastIndex(addr, ":")]
        }
 
        for _, p := range strings.Split(noProxy, ",") {
@@ -1245,7 +1245,7 @@ func useProxy(addr string) bool {
                        continue
                }
                if hasPort(p) {
-                       p = p[:strings.LastIndexByte(p, ':')]
+                       p = p[:strings.LastIndex(p, ":")]
                }
                if addr == p {
                        return false
@@ -1317,7 +1317,7 @@ func (cm *connectMethod) addr() string {
 func (cm *connectMethod) tlsHost() string {
        h := cm.targetAddr
        if hasPort(h) {
-               h = h[:strings.LastIndexByte(h, ':')]
+               h = h[:strings.LastIndex(h, ":")]
        }
        return h
 }
index 903dbd0c4015c22b96d68337bb219f8ada310e63..9539c226974b605f2b17f7c38a6bdd182d9925e8 100644 (file)
@@ -177,7 +177,7 @@ func (p *AddressParser) ParseList(list string) ([]*Address, error) {
 // the name will be rendered according to RFC 2047.
 func (a *Address) String() string {
        // Format address local@domain
-       at := strings.LastIndexByte(a.Address, '@')
+       at := strings.LastIndex(a.Address, "@")
        var local, domain string
        if at < 0 {
                // This is a malformed address ("@" is required in addr-spec);
index 0499f4abe20564189bdf1dc588be35e735c503a4..29aae7ee7ff04b19815cfac7ebaf6b82266435d4 100644 (file)
@@ -592,7 +592,7 @@ func (server *Server) readRequestHeader(codec ServerCodec) (svc *service, mtype
        // we can still recover and move on to the next request.
        keepReading = true
 
-       dot := strings.LastIndexByte(req.ServiceMethod, '.')
+       dot := strings.LastIndex(req.ServiceMethod, ".")
        if dot < 0 {
                err = errors.New("rpc: service/method request ill-formed: " + req.ServiceMethod)
                return
index 53122c79cd93933633844a2d63fb1f6e3db44ebe..c9353ab080d72abaf669ab8874037982a737e8ce 100644 (file)
@@ -532,7 +532,7 @@ func parse(rawurl string, viaRequest bool) (*URL, error) {
 }
 
 func parseAuthority(authority string) (user *Userinfo, host string, err error) {
-       i := strings.LastIndexByte(authority, '@')
+       i := strings.LastIndex(authority, "@")
        if i < 0 {
                host, err = parseHost(authority)
        } else {
@@ -569,7 +569,7 @@ func parseHost(host string) (string, error) {
        if strings.HasPrefix(host, "[") {
                // Parse an IP-Literal in RFC 3986 and RFC 6874.
                // E.g., "[fe80::1]", "[fe80::1%25en0]", "[fe80::1]:80".
-               i := strings.LastIndexByte(host, ']')
+               i := strings.LastIndex(host, "]")
                if i < 0 {
                        return "", errors.New("missing ']' in host")
                }
@@ -885,7 +885,7 @@ func resolvePath(base, ref string) string {
        if ref == "" {
                full = base
        } else if ref[0] != '/' {
-               i := strings.LastIndexByte(base, '/')
+               i := strings.LastIndex(base, "/")
                full = base[:i+1] + ref
        } else {
                full = ref
index 4e1ffb0014660aef79d9757ea4ae07a2f251a59d..793d4d98b3a6aef429196c9f4a5186af131944ae 100644 (file)
@@ -26,7 +26,7 @@ func chkStat(file string) error {
 }
 
 func hasExt(file string) bool {
-       i := strings.LastIndexByte(file, '.')
+       i := strings.LastIndex(file, ".")
        if i < 0 {
                return false
        }
index 074cfff67ae5913a4e38338fb0eedc9ec6fbd309..5c905110a1bd72a4d34e49654d3583c19be5eaf8 100644 (file)
@@ -145,7 +145,7 @@ func Clean(path string) string {
 // file set to path.
 // The returned values have the property that path = dir+file.
 func Split(path string) (dir, file string) {
-       i := strings.LastIndexByte(path, '/')
+       i := strings.LastIndex(path, "/")
        return path[:i+1], path[i+1:]
 }
 
@@ -187,7 +187,7 @@ func Base(path string) string {
                path = path[0 : len(path)-1]
        }
        // Find the last element
-       if i := strings.LastIndexByte(path, '/'); i >= 0 {
+       if i := strings.LastIndex(path, "/"); i >= 0 {
                path = path[i+1:]
        }
        // If empty now, it had only slashes.
index d5c045f25387ad6fe75ae742d26dc780ae068cec..a170cd0fd9a46e222f6288ca80e6eb8ae9120640 100644 (file)
@@ -360,7 +360,7 @@ func (c *common) decorate(s string) string {
        _, file, line, ok := runtime.Caller(skip)
        if ok {
                // Truncate file name at last file name separator.
-               if index := strings.LastIndexByte(file, '/'); index >= 0 {
+               if index := strings.LastIndex(file, "/"); index >= 0 {
                        file = file[index+1:]
                } else if index = strings.LastIndex(file, "\\"); index >= 0 {
                        file = file[index+1:]
index cc24db07f1bff9c8f9446805462895e8b05661d1..8778ab78a03ac4886c07eb8bf1af0d55f4c3bc46 100644 (file)
@@ -253,7 +253,7 @@ func TestMonotonicString(t *testing.T) {
                t1 := Now()
                SetMono(&t1, tt.mono)
                s := t1.String()
-               got := s[strings.LastIndexByte(s, ' ')+1:]
+               got := s[strings.LastIndex(s, " ")+1:]
                if got != tt.want {
                        t.Errorf("with mono=%d: got %q; want %q", tt.mono, got, tt.want)
                }