Skip to content

Commit 0c83fe6

Browse files
committed
chore: resolve linting errors and nil pointer dereference identified in CI
1 parent 7574357 commit 0c83fe6

2 files changed

Lines changed: 11 additions & 6 deletions

File tree

internal/ui/model.go

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -333,13 +333,13 @@ func (m *Model) renderTable() string {
333333
m.config.GoVersion, m.table.GoroutineCount(), uptime,
334334
m.table.Filter, m.config.Readonly, m.config.SHA256, m.pulse)
335335
_, _ = b.WriteString(header)
336-
_, _ = b.WriteString("\n")
336+
_, _ = _, _ = b.WriteString("\n")
337337

338338
// Column headers with sort indicator
339339
sortCol, sortInd := m.table.SortColumnName()
340340
colHeaders := RenderColumnHeaders(m.width, sortCol, sortInd)
341341
_, _ = b.WriteString(colHeaders)
342-
_, _ = b.WriteString("\n")
342+
_, _ = _, _ = b.WriteString("\n")
343343

344344
// Table rows
345345
visibleRows := m.table.VisibleSlice()
@@ -358,15 +358,15 @@ func (m *Model) renderTable() string {
358358
// Check if this row is the selected row by pointer.
359359
selected := (row == m.table.SelectedRow())
360360

361-
b.WriteString(RenderRow(row, m.width, selected))
362-
b.WriteString("\n")
361+
_, _ = b.WriteString(RenderRow(row, m.width, selected))
362+
_, _ = b.WriteString("\n")
363363
rowsRendered++
364364
}
365365

366366
// Pad remaining rows
367367
for rowsRendered < maxRows {
368368
_, _ = b.WriteString(strings.Repeat(" ", m.width))
369-
_, _ = b.WriteString("\n")
369+
_, _ = _, _ = b.WriteString("\n")
370370
rowsRendered++
371371
}
372372

internal/ui/table.go

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -356,5 +356,10 @@ func (t *Table) SaveSnapshot(filename string) error {
356356

357357
encoder := json.NewEncoder(f)
358358
encoder.SetIndent("", " ")
359-
return encoder.Encode(t.AllRows)
359+
err = encoder.Encode(t.AllRows)
360+
if err != nil {
361+
_ = f.Close()
362+
return err
363+
}
364+
return f.Close()
360365
}

0 commit comments

Comments
 (0)