diff options
author | 2018-07-28 17:32:13 +0800 | |
---|---|---|
committer | 2018-07-28 10:32:13 +0100 | |
commit | 54ec78c1ba918b98ac9b9c5b1334c9d170502119 (patch) | |
tree | 65c80cb2c6f886855183532350aedc02e3ca8ee4 /test | |
parent | 62df253371f354200a3efde9e86e80eb50a2441b (diff) | |
download | coredns-54ec78c1ba918b98ac9b9c5b1334c9d170502119.tar.gz coredns-54ec78c1ba918b98ac9b9c5b1334c9d170502119.tar.zst coredns-54ec78c1ba918b98ac9b9c5b1334c9d170502119.zip |
Fix ineffassign (#1959)
* Fix ineffassign
This fix tries to fix ineffassign, as was reported in:
https://goreportcard.com/report/github.com/coredns/coredns#ineffassign
Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
* Update setup.go
Revert this one change, so this can be merged.
Diffstat (limited to 'test')
-rw-r--r-- | test/server_reverse_test.go | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/test/server_reverse_test.go b/test/server_reverse_test.go index 6434d867d..00db410bd 100644 --- a/test/server_reverse_test.go +++ b/test/server_reverse_test.go @@ -74,10 +74,8 @@ func TestReverse(t *testing.T) { m := new(dns.Msg) for i, tc := range tests { - inaddr := tc.addr - var err error if !strings.HasSuffix(tc.addr, ".arpa.") { - inaddr, err = dns.ReverseAddr(tc.addr) + inaddr, err := dns.ReverseAddr(tc.addr) if err != nil { t.Fatalf("Test %d, failed to convert %s", i, tc.addr) } @@ -122,10 +120,8 @@ func TestReverseInAddr(t *testing.T) { m := new(dns.Msg) for i, tc := range tests { - inaddr := tc.addr - var err error if !strings.HasSuffix(tc.addr, ".arpa.") { - inaddr, err = dns.ReverseAddr(tc.addr) + inaddr, err := dns.ReverseAddr(tc.addr) if err != nil { t.Fatalf("Test %d, failed to convert %s", i, tc.addr) } |