Skip to content

Commit c8276f5

Browse files
committed
internal: Fix linters
1 parent 3d068e0 commit c8276f5

File tree

3 files changed

+13
-13
lines changed

3 files changed

+13
-13
lines changed

internal/puzzle/solver.go

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -29,31 +29,31 @@ var (
2929
func Register(name string, solver Solver) {
3030
solversMu.Lock()
3131
defer solversMu.Unlock()
32+
3233
if solver == nil {
3334
panic("puzzle: Register solver is nil")
3435
}
36+
3537
if _, dup := solvers[name]; dup {
3638
panic("puzzle: Register called twice for solver " + name)
3739
}
38-
solvers[name] = solver
39-
}
4040

41-
func unregisterAllSolvers() {
42-
solversMu.Lock()
43-
defer solversMu.Unlock()
44-
// For tests.
45-
solvers = make(map[string]Solver)
41+
solvers[name] = solver
4642
}
4743

4844
// Solvers returns a sorted list of the names of the registered puzzle solvers.
4945
func Solvers() []string {
5046
solversMu.RLock()
5147
defer solversMu.RUnlock()
52-
var list []string
48+
49+
list := make([]string, 0, len(solvers))
50+
5351
for name := range solvers {
5452
list = append(list, name)
5553
}
54+
5655
sort.Strings(list)
56+
5757
return list
5858
}
5959

internal/solutions/day01/fuel.go

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ package day01
33
import (
44
"bufio"
55
"io"
6-
"log"
76
"strconv"
87

98
"github.com/pkg/errors"
@@ -81,24 +80,21 @@ func calc(input io.Reader, calcFn calcFunc) (string, error) {
8180
}
8281
lines++
8382
}
84-
log.Printf("lines: %d\n", lines)
8583

8684
close(in)
8785

8886
if err = scanner.Err(); err != nil {
8987
return "", errors.Wrap(err, "scanner error")
9088
}
91-
var resNumCount int
89+
9290
for lines > 0 {
9391
select {
9492
case r := <-res:
9593
sum += r
96-
resNumCount++
9794
case <-done:
9895
lines--
9996
}
10097
}
101-
log.Printf("res count: %d \n", resNumCount)
10298

10399
close(res)
104100

internal/solutions/day01/fuel_test.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,10 @@ func Test_calcPart1(t *testing.T) {
9393
want: 33583,
9494
},
9595
}
96+
9697
for _, tt := range tests {
98+
tt := tt
99+
97100
t.Run(tt.name, func(t *testing.T) {
98101
in <- module{mass: tt.input}
99102

@@ -149,6 +152,7 @@ func Test_calcPart2(t *testing.T) {
149152
want: 50346,
150153
},
151154
}
155+
152156
for _, tt := range tests {
153157
tt := tt
154158

0 commit comments

Comments
 (0)