refactor: remove redundant nil check in flush (#812)

From the Go docs: "If the map is nil, the number of iterations is 0." [1] Therefore, an additional nil check for before the loop is unnecessary. [1]: https://go.dev/ref/spec#For_range Signed-off-by: Eng Zer Jun <[email protected]>
charmbracelet · Aug 24, 2023 · 12f4057 · 12f4057
1 parent b5e2519
commit 12f4057
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/standard_renderer.go b/standard_renderer.go
@@ -208,10 +208,8 @@ func (r *standardRenderer) flush() {
 
 	// Merge the set of lines we're skipping as a rendering optimization with
 	// the set of lines we've explicitly asked the renderer to ignore.
-	if r.ignoreLines != nil {
-		for k, v := range r.ignoreLines {
-			skipLines[k] = v
-		}
+	for k, v := range r.ignoreLines {
+		skipLines[k] = v
 	}
 
 	// Paint new lines