]> Cypherpunks repositories - gostls13.git/commitdiff
net: use stringslite package for string operations
authoraimuz <mr.imuz@gmail.com>
Fri, 17 May 2024 06:13:32 +0000 (06:13 +0000)
committerGopher Robot <gobot@golang.org>
Sat, 18 May 2024 03:13:26 +0000 (03:13 +0000)
- Replace manual string suffix removal with stringslite.TrimSuffix in conf.go
- Use stringslite.Cut for string splitting in ParseCIDR function in ip.go
- Add stringslite import in ip.go

This change simplifies string operations and improves code readability.

Change-Id: I02c238d0bc91e95789d8060e6ef4c7d4f6e3f0d9
GitHub-Last-Rev: aef5dc5011217abc95b2a2d7c1d991ca84060d59
GitHub-Pull-Request: golang/go#67461
Reviewed-on: https://go-review.googlesource.com/c/go/+/586157
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Auto-Submit: Ian Lance Taylor <iant@golang.org>
Commit-Queue: Ian Lance Taylor <iant@golang.org>
Reviewed-by: Damien Neil <dneil@google.com>
src/net/conf.go
src/net/ip.go

index 7499d49045a52f7feeb06b108aca3130f936ee10..358f5434c4de364ea78ecd9e019a67c8719fda99 100644 (file)
@@ -336,9 +336,7 @@ func (c *conf) lookupOrder(r *Resolver, hostname string) (ret hostLookupOrder, d
        }
 
        // Canonicalize the hostname by removing any trailing dot.
-       if stringslite.HasSuffix(hostname, ".") {
-               hostname = hostname[:len(hostname)-1]
-       }
+       hostname = stringslite.TrimSuffix(hostname, ".")
 
        nss := getSystemNSS()
        srcs := nss.sources["hosts"]
index 6083dd8bf9f60455c9e0e225a5e19bf5add09f7f..49124d95e79df1c933106410491c09d4070ce4b3 100644 (file)
@@ -15,6 +15,7 @@ package net
 import (
        "internal/bytealg"
        "internal/itoa"
+       "internal/stringslite"
        "net/netip"
 )
 
@@ -515,11 +516,10 @@ func parseIP(s string) ([16]byte, bool) {
 // For example, ParseCIDR("192.0.2.1/24") returns the IP address
 // 192.0.2.1 and the network 192.0.2.0/24.
 func ParseCIDR(s string) (IP, *IPNet, error) {
-       i := bytealg.IndexByteString(s, '/')
-       if i < 0 {
+       addr, mask, found := stringslite.Cut(s, "/")
+       if !found {
                return nil, nil, &ParseError{Type: "CIDR address", Text: s}
        }
-       addr, mask := s[:i], s[i+1:]
 
        ipAddr, err := netip.ParseAddr(addr)
        if err != nil || ipAddr.Zone() != "" {