Skip to content

Commit

Permalink
Moved firing of allSessionClosed event to the executor thread to avoi…
Browse files Browse the repository at this point in the history
…d extra threads.
  • Loading branch information
olegstepanov committed Jul 31, 2018
1 parent 4152ce3 commit 1da9990
Showing 1 changed file with 4 additions and 7 deletions.
11 changes: 4 additions & 7 deletions terminal/src/com/jediterm/terminal/ui/JediTermWidget.java
Original file line number Diff line number Diff line change
Expand Up @@ -217,13 +217,7 @@ public TerminalSession getCurrentSession() {

@Override
public TerminalSession createTerminalSession(TtyConnector ttyConnector) {
setTtyConnector(ttyConnector);
new TtyConnectorWaitFor(ttyConnector, Executors.newSingleThreadExecutor()).setTerminationCallback(x -> {
for (TerminalWidgetListener listener : myListeners) {
listener.allSessionsClosed(this);
}
return true;
});
setTtyConnector(ttyConnector);;
return this;
}

Expand Down Expand Up @@ -380,6 +374,9 @@ public void run() {
TerminalPanelListener terminalPanelListener = myTerminalPanel.getTerminalPanelListener();
if (terminalPanelListener != null)
terminalPanelListener.onSessionChanged(getCurrentSession());
for (TerminalWidgetListener listener : myListeners) {
listener.allSessionsClosed(JediTermWidget.this);
}
myTerminalPanel.setKeyListener(myPreConnectHandler);
}
}
Expand Down

0 comments on commit 1da9990

Please sign in to comment.