Add a AbstractTerminal#close() implementation

jline · Jun 14, 2018 · 51c4621 · 51c4621
1 parent cde6119
commit 51c4621
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 3 deletions.
diff --git a/terminal/src/main/java/org/jline/terminal/impl/AbstractPosixTerminal.java b/terminal/src/main/java/org/jline/terminal/impl/AbstractPosixTerminal.java
@@ -72,6 +72,7 @@ public void setSize(Size size) {
     }
 
     public void close() throws IOException {
+        super.close();
         pty.setAttr(originalAttributes);
         pty.close();
     }

diff --git a/terminal/src/main/java/org/jline/terminal/impl/AbstractTerminal.java b/terminal/src/main/java/org/jline/terminal/impl/AbstractTerminal.java
@@ -70,6 +70,9 @@ public void raise(Signal signal) {
         }
     }
 
+    public void close() throws IOException {
+    }
+
     protected void echoSignal(Signal signal) {
         ControlChar cc = null;
         switch (signal) {

diff --git a/terminal/src/main/java/org/jline/terminal/impl/AbstractWindowsTerminal.java b/terminal/src/main/java/org/jline/terminal/impl/AbstractWindowsTerminal.java
@@ -215,6 +215,7 @@ public void setSize(Size size) {
     }
 
     public void close() throws IOException {
+        super.close();
         closing = true;
         pump.interrupt();
         ShutdownHooks.remove(closer);

diff --git a/terminal/src/main/java/org/jline/terminal/impl/DumbTerminal.java b/terminal/src/main/java/org/jline/terminal/impl/DumbTerminal.java
@@ -126,7 +126,4 @@ public void setSize(Size sz) {
         size.copy(sz);
     }
 
-    public void close() throws IOException {
-    }
-
 }
diff --git a/terminal/src/main/java/org/jline/terminal/impl/LineDisciplineTerminal.java b/terminal/src/main/java/org/jline/terminal/impl/LineDisciplineTerminal.java
@@ -234,6 +234,7 @@ protected void processIOException(IOException ioException) {
     }
 
     public void close() throws IOException {
+        super.close();
         try {
             slaveReader.close();
         } finally {