Skip to content

Commit 6db1a88

Browse files
committed
net: fix some ineffectual assignments reported by ineffassign
1 parent 2e509e6 commit 6db1a88

File tree

2 files changed

+10
-8
lines changed

2 files changed

+10
-8
lines changed

src/net/dial_test.go

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1088,14 +1088,16 @@ func TestDialContext(t *testing.T) {
10881088
var c Conn
10891089
switch network {
10901090
case "tcp", "tcp4", "tcp6":
1091-
raddr, err := netip.ParseAddrPort(ln.Addr().String())
1091+
var raddr netip.AddrPort
1092+
raddr, err = netip.ParseAddrPort(ln.Addr().String())
10921093
if err != nil {
10931094
t.Error(err)
10941095
continue
10951096
}
10961097
c, err = d.DialTCP(context.WithValue(context.Background(), "id", i+1), network, (*TCPAddr)(nil).AddrPort(), raddr)
10971098
case "unix", "unixpacket":
1098-
raddr, err := ResolveUnixAddr(network, ln.Addr().String())
1099+
var raddr *UnixAddr
1100+
raddr, err = ResolveUnixAddr(network, ln.Addr().String())
10991101
if err != nil {
11001102
t.Error(err)
11011103
continue
@@ -1131,14 +1133,16 @@ func TestDialContext(t *testing.T) {
11311133
var c2 Conn
11321134
switch network {
11331135
case "udp", "udp4", "udp6":
1134-
raddr, err := netip.ParseAddrPort(c1.LocalAddr().String())
1136+
var raddr netip.AddrPort
1137+
raddr, err = netip.ParseAddrPort(c1.LocalAddr().String())
11351138
if err != nil {
11361139
t.Error(err)
11371140
continue
11381141
}
11391142
c2, err = d.DialUDP(context.WithValue(context.Background(), "id", i+1), network, (*UDPAddr)(nil).AddrPort(), raddr)
11401143
case "unixgram":
1141-
raddr, err := ResolveUnixAddr(network, c1.LocalAddr().String())
1144+
var raddr *UnixAddr
1145+
raddr, err = ResolveUnixAddr(network, c1.LocalAddr().String())
11421146
if err != nil {
11431147
t.Error(err)
11441148
continue

src/net/dnsclient_unix.go

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -495,9 +495,8 @@ func avoidDNS(name string) bool {
495495
// nameList returns a list of names for sequential DNS queries.
496496
func (conf *dnsConfig) nameList(name string) []string {
497497
// Check name length (see isDomainName).
498-
l := len(name)
499-
rooted := l > 0 && name[l-1] == '.'
500-
if l > 254 || l == 254 && !rooted {
498+
rooted := len(name) > 0 && name[len(name)-1] == '.'
499+
if len(name) > 254 || len(name) == 254 && !rooted {
501500
return nil
502501
}
503502

@@ -511,7 +510,6 @@ func (conf *dnsConfig) nameList(name string) []string {
511510

512511
hasNdots := bytealg.CountString(name, '.') >= conf.ndots
513512
name += "."
514-
l++
515513

516514
// Build list of search choices.
517515
names := make([]string, 0, 1+len(conf.search))

0 commit comments

Comments
 (0)