From edf897182ea53d2c941591eae8c1a1534a946323 Mon Sep 17 00:00:00 2001 From: Daniel Imms Date: Fri, 10 Jun 2016 11:51:00 -0700 Subject: [PATCH] Bound refresh max rows to this.rows not this.lines this.row is the size of rows in the viewport, this.lines is the buffer. It's only possible to refresh `0` to `this.rows - 1`. Fixes #86 --- src/xterm.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/xterm.js b/src/xterm.js index 7f01e2a4..8fc154fe 100644 --- a/src/xterm.js +++ b/src/xterm.js @@ -1135,9 +1135,9 @@ width = this.cols; y = start; - if (end >= this.lines.length) { + if (end >= this.rows.length) { this.log('`end` is too large. Most likely a bad CSR.'); - end = this.lines.length - 1; + end = this.rows.length - 1; } for (; y <= end; y++) {