fix(uart): leaking event listener on process exit

Author: stoprocent

lib/uart.js

@@ -25,7 +25,6 @@ class BluetoothHciSocket extends EventEmitter {
     this._queue = null;
     this._parser = null;
     this._exitHandler = this.reset.bind(this);
-    process.on('exit', this._exitHandler);
   }
 
   setFilter (filter) {
@@ -206,6 +205,7 @@ class BluetoothHciSocket extends EventEmitter {
 
     if (!this._serialDevice.isOpen) {
       this._serialDevice.open();
+      process.on('exit', this._exitHandler);
     }
 
     this._hciSerialParser.removeAllListeners('data');
@@ -217,6 +217,9 @@ class BluetoothHciSocket extends EventEmitter {
   }
 
   stop () {
+    // Remove the exit listener
+    process.removeListener('exit', this._exitHandler);
+
     if (this._mode !== 'raw' && this._mode !== 'user') {
       return;
     }
@@ -225,8 +228,6 @@ class BluetoothHciSocket extends EventEmitter {
       this._serialDevice.close();
     }
 
-    process.removeListener('exit', this._exitHandler);
-    
     this._isReconnectionCancelled = true;
     this._hciSerialParser.removeAllListeners('data');
     this._serialDevice.removeAllListeners();