diff --git a/infinitecraft/merger.go b/infinitecraft/merger.go index 1ac8ca7..1cf503a 100644 --- a/infinitecraft/merger.go +++ b/infinitecraft/merger.go @@ -172,6 +172,7 @@ func (m *Merger) getRecipes(ctx context.Context, items []Item) (err error) { return } + //nolint:errcheck // I don't care defer rows.Close() rm := make(map[int32]map[int32][]int32) var found bool @@ -214,11 +215,11 @@ func addRecs(recs *[][]int32, rec []int32) { func recSorter(rec []int32) func([]int32) bool { return func(v []int32) bool { slices.Sort(v) - if !(len(v) == len(rec)) { + if len(v) != len(rec) { return false } for idx, i := range v { - if !(i == rec[idx]) { + if i != rec[idx] { return false } } @@ -256,6 +257,7 @@ func (m *Merger) insertGames(ctx context.Context) (mp map[int64]Game, err error) if err != nil { return } + //nolint:errcheck // I don't care defer stmt.Close() for _, g := range m.games { res, err = stmt.ExecContext(ctx, g) @@ -289,11 +291,13 @@ func (m *Merger) insertItems(ctx context.Context, gameID int64, items []Item) (i if err != nil { return } + //nolint:errcheck // I don't care defer stmt.Close() fetchIDStmt, err = m.db.PreparexContext(ctx, `SELECT id FROM items WHERE name = ?`) if err != nil { return } + //nolint:errcheck // I don't care defer fetchIDStmt.Close() relStmt, err = m.db.PreparexContext(ctx, ` INSERT INTO games_items @@ -303,6 +307,7 @@ func (m *Merger) insertItems(ctx context.Context, gameID int64, items []Item) (i if err != nil { return } + //nolint:errcheck // I don't care defer relStmt.Close() for _, it := range items { _, err = stmt.ExecContext(ctx, it) @@ -334,6 +339,7 @@ func (m *Merger) insertRecipes(ctx context.Context, gameID, itemID int64, recipe if err != nil { return } + //nolint:errcheck // I don't care defer insertStmt.Close() insertRecItemStmt, err = m.db.PreparexContext(ctx, ` INSERT INTO recipes_items @@ -343,6 +349,7 @@ func (m *Merger) insertRecipes(ctx context.Context, gameID, itemID int64, recipe if err != nil { return } + //nolint:errcheck // I don't care defer insertRecItemStmt.Close() selStmt, err = m.db.PreparexContext(ctx, ` SELECT item_id FROM games_items @@ -351,6 +358,7 @@ func (m *Merger) insertRecipes(ctx context.Context, gameID, itemID int64, recipe if err != nil { return } + //nolint:errcheck // I don't care defer selStmt.Close() for _, recipe := range recipes { var recipeID int64 diff --git a/infinitecraft/new.go b/infinitecraft/new.go index c2575ca..bdbe544 100644 --- a/infinitecraft/new.go +++ b/infinitecraft/new.go @@ -14,6 +14,7 @@ func NewMerger(args []string) (*Merger, error) { } fs := make([]*os.File, len(args)) for idx, pth := range args { + //nolint:gosec // Yes, of course fs[idx], err = os.Open(pth) if err != nil { return nil, err diff --git a/infinitecraft/write.go b/infinitecraft/write.go index 8f8e91f..a01c910 100644 --- a/infinitecraft/write.go +++ b/infinitecraft/write.go @@ -14,12 +14,15 @@ func (m *Merger) Write(ctx context.Context, g Game) (out string, err error) { out = base + ".ic" var f *os.File + //nolint:gosec // Yes, of course I'm getting the file from user input if f, err = os.Create(out); err != nil { return } + //nolint:errcheck // I don't care defer f.Close() gz := gzip.NewWriter(f) + //nolint:errcheck // I don't care defer gz.Close() gz.Name = base gz.ModTime = time.Unix(g.Updated/1000, g.Updated%1000)