Switched to the disposable pattern wherever possible.

Author: jsmecham

lib/terminal-session.js

@@ -19,6 +19,8 @@ export default class TerminalSession {
     this.pty = this.openPseudoterminal();
     this.xterm = new Xterm();
 
+    this.disposables.add(this.emitter, this.xterm);
+
     this.handleEvents();
   }
 
@@ -37,7 +39,7 @@ export default class TerminalSession {
     });
 
     // Process Terminal Exit Events
-    this.pty.on('exit', this.destroy.bind(this));
+    this.pty.on('exit', this.dispose.bind(this));
 
   }
 
@@ -88,19 +90,15 @@ export default class TerminalSession {
     };
   }
 
-  destroy() {
+  dispose() {
 
     // Kill the Pseudoterminal (pty) Process
     if (this.pty) this.pty.kill();
 
-    // Dispose of the Terminal Instance
-    if (this.xterm) this.xterm.dispose();
-
-    // Notify any observers that this session is being destroyed.
+    // Notify any observers that this session is being disposed.
     this.emitter.emit('did-destroy', this);
 
     // Clean up any disposables we're responsible for.
-    this.emitter.dispose();
     this.disposables.dispose();
 
   }

lib/terminal-view.js

@@ -18,7 +18,7 @@ export default class TerminalView {
     // Observe the Session to know when it is destroyed so that we can
     // clean up our state (i.e. remove event observers).
     //
-    this.session.onDidDestroy(this.destroy.bind(this));
+    this.session.onDidDestroy(this.dispose.bind(this));
 
     // TODO: Documentation says this should be set for Atom... Research!
     // etch.setScheduler(atom.views);
@@ -38,7 +38,7 @@ export default class TerminalView {
     return etch.update(this);
   }
 
-  destroy() {
+  dispose() {
     this.resizeObserver.disconnect();
     this.disposables.dispose();
     etch.destroy(this);

spec/terminal-session-spec.js

@@ -12,7 +12,7 @@ describe('TerminalSession', () => {
   });
 
   afterEach(() => {
-    testSession.destroy();
+    testSession.dispose();
   });
 
   describe('xterm', () => {

spec/terminal-view-spec.js

@@ -15,8 +15,8 @@ describe('TerminalView', () => {
   });
 
   afterEach(() => {
-    terminalView.destroy();
-    testSession.destroy();
+    terminalView.dispose();
+    testSession.dispose();
   });
 
   describe('focus', () => {