1
1
mirror of https://github.com/walles/moar.git synced 2024-11-22 11:45:50 +03:00
moar/twin/screen.go

786 lines
22 KiB
Go
Raw Permalink Normal View History

// Package twin provides Terminal Window interaction
package twin
import (
"fmt"
"os"
"regexp"
2024-08-12 22:47:46 +03:00
"runtime/debug"
"strconv"
"strings"
"unicode/utf8"
log "github.com/sirupsen/logrus"
"golang.org/x/term"
)
type MouseMode int
const (
MouseModeAuto MouseMode = iota
// Don't capture mouse events. This makes selecting with the mouse work. On
// some terminals mouse scrolling will work using arrow keys emulation, and
// on some not.
MouseModeSelect
// Capture mouse events. This makes mouse scrolling work. Special gymnastics
// will be required for marking with the mouse to copy text.
MouseModeScroll
)
type Screen interface {
// Close() restores terminal to normal state, must be called after you are
// done with your screen
Close()
Clear()
2024-09-21 09:14:36 +03:00
// Returns the width of the rune just added, in number of columns.
//
// Note that if you set a wide rune (like '午') in one column, then whatever
// you put in the next column will be hidden by the wide rune. A wide rune
// in the last screen column will be replaced by a space, to prevent it from
// overflowing onto the next line.
SetCell(column int, row int, styledRune StyledRune) int
// Render our contents into the terminal window
Show()
// Can be called after Close()ing the screen to fake retaining its output.
// Plain Show() is what you'd call during normal operation.
ShowNLines(lineCountToShow int)
// Returns screen width and height.
//
// NOTE: Never cache this response! On window resizes you'll get an
// EventResize on the Screen.Events channel, and this method will start
// returning the new size instead.
Size() (width int, height int)
// ShowCursorAt() moves the cursor to the given screen position and makes
// sure it is visible.
//
// If the position is outside of the screen, the cursor will be hidden.
ShowCursorAt(column int, row int)
// RequestTerminalBackgroundColor() asks the terminal to report its
// background color.
//
// If your terminal supports background color queries and it responds, the
// result will be reported as an EventTerminalBackgroundDetected on the
// Events() channel.
RequestTerminalBackgroundColor()
// This channel is what your main loop should be checking.
Events() chan Event
}
2024-07-14 12:26:10 +03:00
type interruptableReader interface {
Read(p []byte) (n int, err error)
// Interrupt unblocks the read call, either now or eventually.
Interrupt()
}
type UnixScreen struct {
widthAccessFromSizeOnly int // Access from Size() method only
heightAccessFromSizeOnly int // Access from Size() method only
cells [][]StyledRune
// Note that the type here doesn't matter, we only want to know whether or
// not this channel has been signalled
sigwinch chan int
2021-04-17 10:28:45 +03:00
events chan Event
2024-07-14 12:26:10 +03:00
ttyInReader interruptableReader
2021-04-17 10:28:45 +03:00
ttyIn *os.File
2022-02-13 10:34:09 +03:00
oldTerminalState *term.State //nolint Not used on Windows
oldTtyInMode uint32 //nolint Windows only
2021-04-17 08:44:24 +03:00
ttyOut *os.File
2022-02-13 10:34:09 +03:00
oldTtyOutMode uint32 //nolint Windows only
2024-08-12 19:36:49 +03:00
terminalColorCount ColorCount
}
// Example event: "\x1b[<65;127;41M"
//
// Where:
// - "\x1b[<" says this is a mouse event
// - "65" says this is Wheel Up. "64" would be Wheel Down.
// - "127" is the column number on screen, "1" is the first column.
// - "41" is the row number on screen, "1" is the first row.
// - "M" marks the end of the mouse event.
2024-01-01 15:39:30 +03:00
var mouseEventRegex = regexp.MustCompile("^\x1b\\[<([0-9]+);([0-9]+);([0-9]+)M")
// NewScreen() requires Close() to be called after you are done with your new
// screen, most likely somewhere in your shutdown code.
func NewScreen() (Screen, error) {
return NewScreenWithMouseMode(MouseModeAuto)
}
func NewScreenWithMouseMode(mouseMode MouseMode) (Screen, error) {
2024-08-12 19:36:49 +03:00
terminalColorCount := ColorCount24bit
if os.Getenv("COLORTERM") != "truecolor" && strings.Contains(os.Getenv("TERM"), "256") {
// Covers "xterm-256color" as used by the macOS Terminal
2024-08-12 19:36:49 +03:00
terminalColorCount = ColorCount256
}
2024-08-12 19:53:44 +03:00
return NewScreenWithMouseModeAndColorCount(mouseMode, terminalColorCount)
}
2024-08-12 19:53:44 +03:00
func NewScreenWithMouseModeAndColorCount(mouseMode MouseMode, terminalColorCount ColorCount) (Screen, error) {
if !term.IsTerminal(int(os.Stdout.Fd())) {
return nil, fmt.Errorf("stdout (fd=%d) must be a terminal for paging to work", os.Stdout.Fd())
}
screen := UnixScreen{
terminalColorCount: terminalColorCount,
}
// The number "80" here is from manual testing on my MacBook:
//
// First, start "./moar.sh sample-files/large-git-log-patch.txt".
//
// Then do a two finger flick initiating a momentum based scroll-up.
//
// Now, if you get "Events buffer full" warnings, the buffer is too small.
//
// By this definition, 40 was too small, and 80 was OK.
//
// Bumped to 160 because of: https://github.com/walles/moar/issues/164
screen.events = make(chan Event, 160)
screen.setupSigwinchNotification()
err := screen.setupTtyInTtyOut()
if err != nil {
return nil, fmt.Errorf("problem setting up TTY: %w", err)
}
2024-07-14 13:07:30 +03:00
screen.ttyInReader, err = newInterruptableReader(screen.ttyIn)
if err != nil {
restoreErr := screen.restoreTtyInTtyOut()
if restoreErr != nil {
log.Warn("Problem restoring TTY state after failed interruptable reader setup: ", restoreErr)
}
return nil, fmt.Errorf("problem setting up TTY reader: %w", err)
}
screen.setAlternateScreenMode(true)
if mouseMode == MouseModeAuto {
screen.enableMouseTracking(!terminalHasArrowKeysEmulation())
} else if mouseMode == MouseModeSelect {
screen.enableMouseTracking(false)
} else if mouseMode == MouseModeScroll {
screen.enableMouseTracking(true)
} else {
panic(fmt.Errorf("unknown mouse mode: %d", mouseMode))
}
screen.hideCursor(true)
go func() {
defer func() {
2024-08-12 22:47:46 +03:00
panicHandler("NewScreenWithMouseModeAndColorCount()/mainLoop()", recover(), debug.Stack())
}()
screen.mainLoop()
}()
return &screen, nil
}
// Close() restores terminal to normal state, must be called after you are done
// with the screen returned by NewScreen()
func (screen *UnixScreen) Close() {
// Tell the pager to exit unless it hasn't already
screen.events <- EventExit{}
2024-07-14 12:26:10 +03:00
// Tell our main loop to exit
screen.ttyInReader.Interrupt()
screen.hideCursor(false)
screen.enableMouseTracking(false)
screen.setAlternateScreenMode(false)
2021-04-20 09:43:37 +03:00
err := screen.restoreTtyInTtyOut()
if err != nil {
// Debug logging because this is expected to fail in some cases:
// * https://github.com/walles/moar/issues/145
// * https://github.com/walles/moar/issues/149
// * https://github.com/walles/moar/issues/150
log.Debug("Problem restoring TTY state: ", err)
2021-04-20 09:43:37 +03:00
}
}
func (screen *UnixScreen) Events() chan Event {
return screen.events
}
// Write string to ttyOut, panic on failure, return number of bytes written.
func (screen *UnixScreen) write(string string) int {
bytesWritten, err := screen.ttyOut.Write([]byte(string))
if err != nil {
panic(err)
}
return bytesWritten
}
func (screen *UnixScreen) setAlternateScreenMode(enable bool) {
// Ref: https://stackoverflow.com/a/11024208/473672
if enable {
screen.write("\x1b[?1049h")
} else {
screen.write("\x1b[?1049l")
}
}
func (screen *UnixScreen) hideCursor(hide bool) {
// Ref: https://en.wikipedia.org/wiki/ANSI_escape_code#CSI_(Control_Sequence_Introducer)_sequences
if hide {
screen.write("\x1b[?25l")
} else {
screen.write("\x1b[?25h")
}
}
// Some terminals convert mouse events to key events making scrolling better
// without our built-in mouse support, and some do not.
//
// For those that do, we're better off without mouse tracking.
//
// To test your terminal, run with `moar --mousemode=mark` and see if mouse
// scrolling still works (both down and then back up to the top). If it does,
// add another check to this function!
//
// See also: https://github.com/walles/moar/issues/53
func terminalHasArrowKeysEmulation() bool {
// Untested:
// * The Windows terminal
// Better off with mouse tracking:
// * iTerm2 (macOS)
// * Terminal.app (macOS)
2023-12-18 21:52:10 +03:00
// * Contour, thanks to @postsolar (GitHub username) for testing, 2023-12-18
// Hyper, tested on macOS, December 14th 2023
if os.Getenv("TERM_PROGRAM") == "Hyper" {
return true
}
// Kitty, tested on macOS, December 14th 2023
if os.Getenv("KITTY_WINDOW_ID") != "" {
return true
}
// Alacritty, tested on macOS, December 14th 2023
if os.Getenv("ALACRITTY_WINDOW_ID") != "" {
return true
}
// Warp, tested on macOS, December 14th 2023
if os.Getenv("TERM_PROGRAM") == "WarpTerminal" {
return true
}
// GNOME Terminal, tested on Ubuntu 22.04, December 16th 2023
if os.Getenv("GNOME_TERMINAL_SCREEN") != "" {
return true
}
// Tilix, tested on Ubuntu 22.04, December 16th 2023
if os.Getenv("TILIX_ID") != "" {
return true
}
// Konsole, tested on Ubuntu 22.04, December 16th 2023
if os.Getenv("KONSOLE_VERSION") != "" {
return true
}
// Terminator, tested on Ubuntu 22.04, December 16th 2023
if os.Getenv("TERMINATOR_UUID") != "" {
return true
}
// Foot, tested on Ubuntu 22.04, December 16th 2023
2023-12-18 11:53:31 +03:00
if os.Getenv("TERM") == "foot" || strings.HasPrefix(os.Getenv("TERM"), "foot-") {
// Note that this test isn't very good, somebody could be running Foot
// with some other TERM setting. Other suggestions welcome.
return true
}
2024-01-03 20:33:22 +03:00
// Wezterm, tested on MacOS 12.6, January 3rd, 2024
if os.Getenv("TERM_PROGRAM") == "WezTerm" {
return true
}
2024-01-27 01:30:54 +03:00
// Rio, tested on macOS 14.3, January 27th, 2024
if os.Getenv("TERM_PROGRAM") == "rio" {
return true
}
// VSCode 1.89.0, tested on macOS 14.4, May 6th, 2024
if os.Getenv("TERM_PROGRAM") == "vscode" {
return true
}
// IntelliJ IDEA CE 2023.2.2, tested on macOS 14.4, May 6th, 2024
if os.Getenv("TERM_PROGRAM") == "JetBrains-JediTerm" {
return true
}
return false
}
func (screen *UnixScreen) enableMouseTracking(enable bool) {
if enable {
screen.write("\x1b[?1006;1000h")
} else {
screen.write("\x1b[?1006;1000l")
}
}
// ShowCursorAt() moves the cursor to the given screen position and makes sure
// it is visible.
//
// If the position is outside of the screen, the cursor will be hidden.
func (screen *UnixScreen) ShowCursorAt(column int, row int) {
if column < 0 {
screen.hideCursor(true)
return
}
if row < 0 {
screen.hideCursor(true)
return
}
width, height := screen.Size()
if column >= width {
screen.hideCursor(true)
return
}
if row >= height {
screen.hideCursor(true)
return
}
// https://en.wikipedia.org/wiki/ANSI_escape_code#CSI_(Control_Sequence_Introducer)_sequences
screen.write(fmt.Sprintf("\x1b[%d;%dH", row, column))
screen.hideCursor(false)
}
func (screen *UnixScreen) mainLoop() {
defer log.Debug("Twin screen main loop done")
// "1400" comes from me trying fling scroll operations on my MacBook
// trackpad and looking at the high watermark (logged below).
//
// The highest I saw when I tried this was 700 something. 1400 is twice
// that, so 1400 should be good.
buffer := make([]byte, 1400)
maxBytesRead := 0
expectingTerminalBackgroundColor := true
for {
2024-07-14 12:26:10 +03:00
count, err := screen.ttyInReader.Read(buffer)
if err != nil {
// Ref:
// * https://github.com/walles/moar/issues/145
// * https://github.com/walles/moar/issues/149
// * https://github.com/walles/moar/issues/150
log.Debug("ttyin read error, twin giving up: ", err)
screen.events <- EventExit{}
return
}
if expectingTerminalBackgroundColor {
// This is the response to our background color request
bg := parseTerminalBgColorResponse(buffer[0:count])
if bg != nil {
select {
case screen.events <- EventTerminalBackgroundDetected{Color: *bg}:
// Yay
default:
// If this happens, consider increasing the channel size in
// NewScreen()
log.Debugf("Unable to post terminal background color detected event")
}
expectingTerminalBackgroundColor = false
continue
}
}
// We only expect this on entry, it's requested right before we start
2024-08-12 19:53:44 +03:00
// the main loop in NewScreenWithMouseModeAndColorCount().
expectingTerminalBackgroundColor = false
if count > maxBytesRead {
maxBytesRead = count
log.Trace("ttyin high watermark bumped to ", maxBytesRead, " bytes")
}
encodedKeyCodeSequences := string(buffer[0:count])
if !utf8.ValidString(encodedKeyCodeSequences) {
log.Warn("Got invalid UTF-8 sequence on ttyin: ", encodedKeyCodeSequences)
continue
}
for len(encodedKeyCodeSequences) > 0 {
var event *Event
event, encodedKeyCodeSequences = consumeEncodedEvent(encodedKeyCodeSequences)
if event == nil {
// No event, go wait for more
break
}
// Post the event
select {
case screen.events <- *event:
// Yay
default:
// If this happens, consider increasing the channel size in
// NewScreen()
log.Debugf("Events buffer (size %d) full, events are being dropped", cap(screen.events))
}
}
}
}
2023-06-06 09:50:42 +03:00
// Turn ESC into <0x1b> and other low ASCII characters into <0xXX> for logging
// purposes.
2024-01-01 15:39:30 +03:00
func humanizeLowASCII(withLowAsciis string) string {
2023-06-06 09:50:42 +03:00
humanized := ""
for _, char := range withLowAsciis {
if char < ' ' {
humanized += fmt.Sprintf("<0x%2x>", char)
continue
}
humanized += string(char)
}
return humanized
}
// Consume initial key code from the sequence of encoded keycodes.
//
// Returns a (possibly nil) event that should be posted, and the remainder of
// the encoded events sequence.
func consumeEncodedEvent(encodedEventSequences string) (*Event, string) {
for singleKeyCodeSequence, keyCode := range escapeSequenceToKeyCode {
if !strings.HasPrefix(encodedEventSequences, singleKeyCodeSequence) {
continue
}
// Encoded key code sequence found, report it!
var event Event = EventKeyCode{keyCode}
return &event, strings.TrimPrefix(encodedEventSequences, singleKeyCodeSequence)
}
2024-01-01 15:39:30 +03:00
mouseMatch := mouseEventRegex.FindStringSubmatch(encodedEventSequences)
if mouseMatch != nil {
if mouseMatch[1] == "64" {
var event Event = EventMouse{buttons: MouseWheelUp}
2023-01-11 12:44:24 +03:00
return &event, strings.TrimPrefix(encodedEventSequences, mouseMatch[0])
}
if mouseMatch[1] == "65" {
var event Event = EventMouse{buttons: MouseWheelDown}
return &event, strings.TrimPrefix(encodedEventSequences, mouseMatch[0])
}
2023-06-06 09:50:42 +03:00
log.Debug(
"Unhandled multi character mouse escape sequence(s): {",
2024-01-01 15:39:30 +03:00
humanizeLowASCII(encodedEventSequences),
2023-06-06 09:50:42 +03:00
"}")
return nil, ""
}
// No escape sequence prefix matched
runes := []rune(encodedEventSequences)
2023-01-11 13:06:38 +03:00
if len(runes) == 0 {
return nil, ""
}
if runes[0] == '\x1b' {
2023-01-11 13:06:38 +03:00
if len(runes) != 1 {
// This means one or more sequences should be added to
// escapeSequenceToKeyCode in keys.go.
2023-06-06 09:50:42 +03:00
log.Debug(
"Unhandled multi character terminal escape sequence(s): {",
2024-01-01 15:39:30 +03:00
humanizeLowASCII(encodedEventSequences),
2023-06-06 09:50:42 +03:00
"}")
2023-01-11 13:06:38 +03:00
// Mark everything as consumed since we don't know how to proceed otherwise.
return nil, ""
}
var event Event = EventKeyCode{KeyEscape}
return &event, string(runes[1:])
}
if runes[0] == '\r' {
var event Event = EventKeyCode{KeyEnter}
return &event, string(runes[1:])
}
2023-01-11 13:06:38 +03:00
// Report the single rune
var event Event = EventRune{rune: runes[0]}
return &event, string(runes[1:])
}
// Returns screen width and height.
//
// NOTE: Never cache this response! On window resizes you'll get an EventResize
// on the Screen.Events channel, and this method will start returning the new
// size instead.
func (screen *UnixScreen) Size() (width int, height int) {
select {
case <-screen.sigwinch:
// Resize logic needed, see below
default:
// No resize, go with the existing values
return screen.widthAccessFromSizeOnly, screen.heightAccessFromSizeOnly
}
// Window was resized
width, height, err := term.GetSize(int(screen.ttyOut.Fd()))
if err != nil {
panic(err)
}
if screen.widthAccessFromSizeOnly == width && screen.heightAccessFromSizeOnly == height {
// Not sure when this would happen, but if it does this wasn't really a
// resize, and we don't need to treat it as such.
return screen.widthAccessFromSizeOnly, screen.heightAccessFromSizeOnly
}
newCells := make([][]StyledRune, height)
for rowNumber := 0; rowNumber < height; rowNumber++ {
newCells[rowNumber] = make([]StyledRune, width)
}
// FIXME: Copy any existing contents over to the new, resized screen array
// FIXME: Fill any non-initialized cells with whitespace
screen.widthAccessFromSizeOnly = width
screen.heightAccessFromSizeOnly = height
screen.cells = newCells
return screen.widthAccessFromSizeOnly, screen.heightAccessFromSizeOnly
}
func (screen *UnixScreen) RequestTerminalBackgroundColor() {
// Ref:
// https://stackoverflow.com/questions/2507337/how-to-determine-a-terminals-background-color
fmt.Println("\x1b]11;?\x07")
}
func parseTerminalBgColorResponse(responseBytes []byte) *Color {
prefix := "\x1b]11;rgb:"
suffix1 := "\x07"
suffix2 := "\x1b\\"
sampleResponse1 := prefix + "0000/0000/0000" + suffix1
sampleResponse2 := prefix + "0000/0000/0000" + suffix2
if len(responseBytes) != len(sampleResponse1) && len(responseBytes) != len(sampleResponse2) {
// Not a bg color response
return nil
}
response := string(responseBytes)
if !strings.HasPrefix(response, prefix) {
log.Debug("Got unexpected prefix in bg color response from terminal: ", string(responseBytes))
return nil
}
response = strings.TrimPrefix(response, prefix)
if !strings.HasSuffix(response, suffix1) && !strings.HasSuffix(response, suffix2) {
log.Debug("Got unexpected suffix in bg color response from terminal: ", string(responseBytes))
return nil
}
response = strings.TrimSuffix(response, suffix1)
response = strings.TrimSuffix(response, suffix2)
// response is now "RRRR/GGGG/BBBB"
red, err := strconv.ParseUint(response[0:4], 16, 16)
if err != nil {
log.Debug("Failed parsing red in bg color response from terminal: ", string(responseBytes), ": ", err)
return nil
}
green, err := strconv.ParseUint(response[5:9], 16, 16)
if err != nil {
log.Debug("Failed parsing green in bg color response from terminal: ", string(responseBytes), ": ", err)
return nil
}
blue, err := strconv.ParseUint(response[10:14], 16, 16)
if err != nil {
log.Debug("Failed parsing blue in bg color response from terminal: ", string(responseBytes), ": ", err)
return nil
}
color := NewColor24Bit(uint8(red/256), uint8(green/256), uint8(blue/256))
return &color
}
func (screen *UnixScreen) SetCell(column int, row int, styledRune StyledRune) int {
if column < 0 {
return styledRune.Width()
}
if row < 0 {
return styledRune.Width()
}
width, height := screen.Size()
if column >= width {
return styledRune.Width()
}
if row >= height {
return styledRune.Width()
}
if column+styledRune.Width() > width {
// This cell is too wide for the screen, write a space instead
screen.cells[row][column] = NewStyledRune(' ', styledRune.Style)
return styledRune.Width()
}
screen.cells[row][column] = styledRune
return styledRune.Width()
}
func (screen *UnixScreen) Clear() {
empty := NewStyledRune(' ', StyleDefault)
width, height := screen.Size()
for row := 0; row < height; row++ {
for column := 0; column < width; column++ {
screen.cells[row][column] = empty
}
}
}
// A cell is considered hidden if it's preceded by a wide character that spans
// multiple columns.
func withoutHiddenRunes(runes []StyledRune) []StyledRune {
result := make([]StyledRune, 0, len(runes))
for i := 0; i < len(runes); i++ {
if i > 0 && runes[i-1].Width() == 2 {
// This is a hidden rune
continue
}
result = append(result, runes[i])
}
return result
}
// Returns the rendered line, plus how many information carrying cells went into
// it. The width is used to decide whether or not to clear to EOL at the end of
// the line.
func renderLine(row []StyledRune, width int, terminalColorCount ColorCount) (string, int) {
row = withoutHiddenRunes(row)
// Strip trailing whitespace
2023-10-10 22:26:43 +03:00
lastSignificantCellIndex := len(row) - 1
for ; lastSignificantCellIndex >= 0; lastSignificantCellIndex-- {
lastCell := row[lastSignificantCellIndex]
if lastCell.Rune != ' ' || lastCell.Style != StyleDefault {
break
}
}
2023-10-10 22:26:43 +03:00
row = row[0 : lastSignificantCellIndex+1]
var builder strings.Builder
// Set initial line style to normal
builder.WriteString("\x1b[m")
lastStyle := StyleDefault
2023-10-10 22:26:43 +03:00
for column := 0; column < len(row); column++ {
cell := row[column]
style := cell.Style
runeToWrite := cell.Rune
if !Printable(runeToWrite) {
// Highlight unprintable runes
style = Style{
fg: NewColor16(7), // White
bg: NewColor16(1), // Red
attrs: AttrBold,
}
runeToWrite = '?'
}
if style != lastStyle {
builder.WriteString(style.RenderUpdateFrom(lastStyle, terminalColorCount))
lastStyle = style
}
builder.WriteRune(runeToWrite)
}
if len(row) < width {
// Clear to end of line
// https://en.wikipedia.org/wiki/ANSI_escape_code#CSI_(Control_Sequence_Introducer)_sequences
//
// Note that we can't do this if we're one the last screen column:
// https://github.com/microsoft/terminal/issues/18115#issuecomment-2448054645
builder.WriteString(StyleDefault.RenderUpdateFrom(lastStyle, terminalColorCount))
builder.WriteString("\x1b[K")
}
2023-10-10 22:26:43 +03:00
return builder.String(), len(row)
}
func (screen *UnixScreen) Show() {
width, height := screen.Size()
screen.showNLines(width, height, true)
}
func (screen *UnixScreen) ShowNLines(height int) {
width, _ := screen.Size()
screen.showNLines(width, height, false)
}
func (screen *UnixScreen) showNLines(width int, height int, clearFirst bool) {
var builder strings.Builder
if clearFirst {
// Start in the top left corner:
// https://en.wikipedia.org/wiki/ANSI_escape_code#CSI_(Control_Sequence_Introducer)_sequences
builder.WriteString("\x1b[1;1H")
}
for row := 0; row < height; row++ {
rendered, lineLength := renderLine(screen.cells[row], width, screen.terminalColorCount)
builder.WriteString(rendered)
wasLastLine := row == (height - 1)
// NOTE: This <= should *really* be <= and nothing else. Otherwise, if
// one line precisely as long as the terminal window goes before one
// empty line, the empty line will never be rendered.
//
// Can be demonstrated using "moar m/pager.go", scroll right once to
// make the line numbers go away, then make the window narrower until
// some line before an empty line is just as wide as the window.
//
// With the wrong comparison here, then the empty line just disappears.
if lineLength <= len(screen.cells[row]) && !wasLastLine {
builder.WriteString("\r\n")
}
}
// Write out what we have
screen.write(builder.String())
}