1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

inspector: removing checking of non existent field in lib/inspector.js

Browse Source 
Seems like sessionAttached doesn't exist and no more assigned anywhere

PR-URL: https://github.com/nodejs/node/pull/27919
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Eugene Ostroukhov <eostroukhov@google.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
This commit is contained in:
Keroosha 2019-05-26 21:30:10 +03:00 committed by Ujjwal Sharma
parent ff66e08bce
commit c91a90be73
No known key found for this signature in database
GPG Key ID: 1FD3B47B83F46621
1 changed files with 1 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
lib/inspector.js
View File

@ -37,12 +37,8 @@ class Session extends EventEmitter {
connect() { connect() {
if (this[connectionSymbol]) if (this[connectionSymbol])
throw new ERR_INSPECTOR_ALREADY_CONNECTED('The inspector session'); throw new ERR_INSPECTOR_ALREADY_CONNECTED('The inspector session');
const connection = this[connectionSymbol] =
new Connection((message) => this[onMessageSymbol](message)); new Connection((message) => this[onMessageSymbol](message));
if (connection.sessionAttached) {
throw new ERR_INSPECTOR_ALREADY_CONNECTED('Another inspector session');
}
this[connectionSymbol] = connection;
} }
[onMessageSymbol](message) { [onMessageSymbol](message) {