Possible endless loop on windows when the event reading loop fails, f…

…ixes #51
jline · Dec 5, 2016 · 3076c57 · 3076c57
1 parent 5e790b1
commit 3076c57
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 17 deletions.
diff --git a/src/main/java/org/jline/terminal/impl/AbstractWindowsTerminal.java b/src/main/java/org/jline/terminal/impl/AbstractWindowsTerminal.java
@@ -173,7 +173,7 @@ public void close() throws IOException {
         writer.close();
     }
 
-    protected abstract byte[] readConsoleInput();
+    protected abstract byte[] readConsoleInput() throws IOException;
 
     protected String getEscapeSequence(short keyCode) {
         String escapeSequence = null;

diff --git a/src/main/java/org/jline/terminal/impl/jansi/JansiWinSysTerminal.java b/src/main/java/org/jline/terminal/impl/jansi/JansiWinSysTerminal.java
@@ -54,14 +54,8 @@ public Size getSize() {
         return size;
     }
 
-    protected byte[] readConsoleInput() {
-        // XXX does how many events to read in one call matter?
-        INPUT_RECORD[] events = null;
-        try {
-            events = WindowsSupport.readConsoleInput(1);
-        } catch (IOException e) {
-            Log.debug("read Windows terminal input error: ", e);
-        }
+    protected byte[] readConsoleInput() throws IOException {
+        INPUT_RECORD[] events = WindowsSupport.readConsoleInput(1);
         if (events == null) {
             return new byte[0];
         }

diff --git a/src/main/java/org/jline/terminal/impl/jna/win/JnaWinSysTerminal.java b/src/main/java/org/jline/terminal/impl/jna/win/JnaWinSysTerminal.java
@@ -55,14 +55,8 @@ public Size getSize() {
         return new Size(info.windowWidth(), info.windowHeight());
     }
 
-    protected byte[] readConsoleInput() {
-        // XXX does how many events to read in one call matter?
-        Kernel32.INPUT_RECORD[] events = null;
-        try {
-            events = doReadConsoleInput();
-        } catch (IOException e) {
-            Log.debug("read Windows terminal input error: ", e);
-        }
+    protected byte[] readConsoleInput() throws IOException {
+        Kernel32.INPUT_RECORD[] events = doReadConsoleInput();
         if (events == null) {
             return new byte[0];
         }