Skip to content

Commit b57b7c2

Browse files
committed
fix linter
1 parent 26a3439 commit b57b7c2

File tree

3 files changed

+24
-17
lines changed

3 files changed

+24
-17
lines changed

client/request.go

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -729,9 +729,9 @@ type QueryParam struct {
729729
}
730730

731731
// Keys method returns all keys in the query params.
732-
func (f *QueryParam) Keys() []string {
733-
keys := make([]string, f.Len())
734-
f.VisitAll(func(key, value []byte) {
732+
func (p *QueryParam) Keys() []string {
733+
keys := make([]string, 0, p.Len())
734+
p.VisitAll(func(key, _ []byte) {
735735
keys = append(keys, utils.UnsafeString(key))
736736
})
737737

@@ -870,8 +870,8 @@ type FormData struct {
870870

871871
// Keys method returns all keys in the form data.
872872
func (f *FormData) Keys() []string {
873-
keys := make([]string, f.Len())
874-
f.VisitAll(func(key, value []byte) {
873+
keys := make([]string, 0, f.Len())
874+
f.VisitAll(func(key, _ []byte) {
875875
keys = append(keys, utils.UnsafeString(key))
876876
})
877877

client/request_test.go

Lines changed: 15 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -187,7 +187,9 @@ func Benchmark_Request_Headers(b *testing.B) {
187187
b.ReportAllocs()
188188

189189
for i := 0; i < b.N; i++ {
190-
for range req.Headers() {
190+
for k, v := range req.Headers() {
191+
_ = k
192+
_ = v
191193
}
192194
}
193195
}
@@ -346,7 +348,9 @@ func Benchmark_Request_Params(b *testing.B) {
346348
b.ReportAllocs()
347349

348350
for i := 0; i < b.N; i++ {
349-
for range req.Params() {
351+
for k, v := range req.Params() {
352+
_ = k
353+
_ = v
350354
}
351355
}
352356
}
@@ -461,7 +465,9 @@ func Benchmark_Request_Cookies(b *testing.B) {
461465
b.ReportAllocs()
462466

463467
for i := 0; i < b.N; i++ {
464-
for range req.Cookies() {
468+
for k, v := range req.Cookies() {
469+
_ = k
470+
_ = v
465471
}
466472
}
467473
}
@@ -571,7 +577,9 @@ func Benchmark_Request_PathParams(b *testing.B) {
571577
b.ReportAllocs()
572578

573579
for i := 0; i < b.N; i++ {
574-
for range req.PathParams() {
580+
for k, v := range req.PathParams() {
581+
_ = k
582+
_ = v
575583
}
576584
}
577585
}
@@ -1379,7 +1387,9 @@ func Benchmark_Request_FormDatas(b *testing.B) {
13791387
b.ReportAllocs()
13801388

13811389
for i := 0; i < b.N; i++ {
1382-
for range req.FormDatas() {
1390+
for k, v := range req.FormDatas() {
1391+
_ = k
1392+
_ = v
13831393
}
13841394
}
13851395
}

client/response_test.go

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@ import (
44
"bytes"
55
"crypto/tls"
66
"encoding/xml"
7-
"fmt"
87
"io"
98
"net"
109
"os"
@@ -224,11 +223,7 @@ func Test_Response_Headers(t *testing.T) {
224223

225224
headers := make(map[string][]string)
226225
for k, v := range resp.Headers() {
227-
headers[k] = make([]string, 0)
228-
for _, value := range v {
229-
fmt.Print(string(value))
230-
headers[k] = append(headers[k], string(value))
231-
}
226+
headers[k] = append(headers[k], v...)
232227
}
233228

234229
require.Contains(t, headers["Foo"], "bar")
@@ -270,7 +265,9 @@ func Benchmark_Headers(b *testing.B) {
270265
SetClient(client).
271266
Get("http://example.com")
272267

273-
for range resp.Headers() {
268+
for k, v := range resp.Headers() {
269+
_ = k
270+
_ = v
274271
}
275272

276273
resp.Close()

0 commit comments

Comments
 (0)