Simplify code a bit

LICENSE

@@ -1,4 +1,4 @@
-Copyright (c) 2015–2020 Rob Lillack
+Copyright (c) 2015–2021 Rob Lillack
 Copyright (C) 2012 Rob Figueiredo
 All Rights Reserved.
 

binder.go

@@ -329,7 +329,7 @@ func unbindSlice(output map[string]string, name string, val interface{}) {
 func bindStruct(params *Params, name string, typ reflect.Type) reflect.Value {
 	result := reflect.New(typ).Elem()
 	fieldValues := make(map[string]reflect.Value)
-	for key, _ := range params.Values {
+	for key := range params.Values {
 		if !strings.HasPrefix(key, name+".") {
 			continue
 		}

binder_test.go

@@ -274,28 +274,28 @@ var unbinderTestCases = map[string]interface{}{
 // Some of the unbinding results are not exactly what is in PARAMS, since it
 // serializes implicit zero values explicitly.
 var unbinderOverrideAnswers = map[string]map[string]string{
-	"arr": map[string]string{
+	"arr": {
 		"arr[0]": "1",
 		"arr[1]": "2",
 		"arr[2]": "0",
 		"arr[3]": "3",
 	},
-	"A": map[string]string{
+	"A": {
 		"A.ID":      "123",
 		"A.Name":    "rob",
 		"A.B.Extra": "",
 	},
-	"arrC": map[string]string{
+	"arrC": {
 		"arrC[0].ID":      "5",
 		"arrC[0].Name":    "rob",
 		"arrC[0].B.Extra": "foo",
 		"arrC[1].ID":      "8",
 		"arrC[1].Name":    "bill",
 		"arrC[1].B.Extra": "",
 	},
-	"m":  map[string]string{"m[a]": "foo", "m[b]": "bar"},
-	"m2": map[string]string{"m2[1]": "foo", "m2[2]": "bar"},
-	"m3": map[string]string{"m3[a]": "1", "m3[b]": "2"},
+	"m":  {"m[a]": "foo", "m[b]": "bar"},
+	"m2": {"m2[1]": "foo", "m2[2]": "bar"},
+	"m3": {"m3[a]": "1", "m3[b]": "2"},
 }
 
 func TestUnbinder(t *testing.T) {

cert_test.go

@@ -8,8 +8,8 @@ import (
 
 func TestCertificateCreation(t *testing.T) {
 	for org, domains := range map[string][]string{
-		"ACME Inc.": []string{"acme.com", "acme.biz"},
-		"Me":        []string{"::1", "127.0.0.1"},
+		"ACME Inc.": {"acme.com", "acme.biz"},
+		"Me":        {"::1", "127.0.0.1"},
 	} {
 		keypair, err := createCertificate(org, strings.Join(domains, ", "))
 		if err != nil {

fakeapp_test.go

@@ -64,19 +64,19 @@ func (c MyStatic) Serve(prefix, filepath string) Result {
 func startFakeBookingApp() {
 	RegisterController((*Hotels)(nil),
 		[]*MethodType{
-			&MethodType{
+			{
 				Name: "Index",
 			},
-			&MethodType{
+			{
 				Name: "Boom",
 			},
-			&MethodType{
+			{
 				Name: "Show",
 				Args: []*MethodArg{
 					{"id", reflect.TypeOf((*int)(nil))},
 				},
 			},
-			&MethodType{
+			{
 				Name: "Book",
 				Args: []*MethodArg{
 					{"id", reflect.TypeOf((*int)(nil))},
@@ -86,11 +86,11 @@ func startFakeBookingApp() {
 
 	RegisterController((*Static)(nil),
 		[]*MethodType{
-			&MethodType{
+			{
 				Name: "Serve",
 				Args: []*MethodArg{
-					&MethodArg{Name: "prefix", Type: reflect.TypeOf((*string)(nil))},
-					&MethodArg{Name: "filepath", Type: reflect.TypeOf((*string)(nil))},
+					{Name: "prefix", Type: reflect.TypeOf((*string)(nil))},
+					{Name: "filepath", Type: reflect.TypeOf((*string)(nil))},
 				},
 			},
 		})

i18n.go

@@ -31,7 +31,7 @@ var (
 func MessageLanguages() []string {
 	languages := make([]string, len(messages))
 	i := 0
-	for language, _ := range messages {
+	for language := range messages {
 		languages[i] = language
 		i++
 	}

internal/pathtree/tree_test.go

@@ -182,7 +182,7 @@ func BenchmarkTree100(b *testing.B) {
 	b.ResetTimer()
 
 	for i := 0; i < b.N/len(queries); i++ {
-		for k, _ := range queries {
+		for k := range queries {
 			n.Find(k)
 		}
 	}