Bläddra i källkod

Merge pull request #962 from gtardif/fix_panic_terminal_size_debug

Avoid panic in some weird cases (debug in VSCode for example)
Guillaume Tardif 5 år sedan
förälder
incheckning
efdf80ebb8
1 ändrade filer med 2 tillägg och 1 borttagningar
  1. 2 1
      progress/tty.go

+ 2 - 1
progress/tty.go

@@ -150,7 +150,8 @@ func lineText(event Event, terminalWidth, statusPadding int, color bool) string
 	// is 2-3 lines long and breaks the line formating
 	maxStatusLen := terminalWidth - textLen - statusPadding - 15
 	status := event.StatusText
-	if len(status) > maxStatusLen {
+	// in some cases (debugging under VS Code), terminalWidth is set to zero by goterm.Width() ; ensuring we don't tweak strings with negative char index
+	if maxStatusLen > 0 && len(status) > maxStatusLen {
 		status = status[:maxStatusLen] + "..."
 	}
 	text := fmt.Sprintf(" %s %s %s%s %s",