Skip to content

Commit

Permalink
debugger: rename internal library for clarity
Browse files Browse the repository at this point in the history
When I moved these files from node-inspect to Node.js core, I put them
in lib/internal/inspector. That was a mistake. They should be in
lib/internal/debugger.

PR-URL: nodejs#39080
Backport-PR-URL: nodejs#39446
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
  • Loading branch information
Trott authored and foxxyz committed Oct 18, 2021
1 parent 99cb640 commit 0efdeea
Show file tree
Hide file tree
Showing 5 changed files with 6 additions and 6 deletions.
Expand Up @@ -43,8 +43,8 @@ const console = require('internal/console/global');

const { 0: InspectClient, 1: createRepl } =
[
require('internal/inspector/inspect_client'),
require('internal/inspector/inspect_repl'),
require('internal/debugger/inspect_client'),
require('internal/debugger/inspect_repl'),
];

const debuglog = util.debuglog('inspect');
Expand Down
File renamed without changes.
File renamed without changes.
2 changes: 1 addition & 1 deletion lib/internal/main/inspect.js
Expand Up @@ -18,5 +18,5 @@ markBootstrapComplete();

// Start the debugger agent.
process.nextTick(() => {
require('internal/inspector/_inspect').start();
require('internal/debugger/_inspect').start();
});
6 changes: 3 additions & 3 deletions node.gyp
Expand Up @@ -125,6 +125,9 @@
'lib/internal/crypto/sig.js',
'lib/internal/crypto/util.js',
'lib/internal/constants.js',
'lib/internal/debugger/_inspect.js',
'lib/internal/debugger/inspect_client.js',
'lib/internal/debugger/inspect_repl.js',
'lib/internal/dgram.js',
'lib/internal/dns/promises.js',
'lib/internal/dns/utils.js',
Expand All @@ -148,9 +151,6 @@
'lib/internal/heap_utils.js',
'lib/internal/histogram.js',
'lib/internal/idna.js',
'lib/internal/inspector/_inspect.js',
'lib/internal/inspector/inspect_client.js',
'lib/internal/inspector/inspect_repl.js',
'lib/internal/inspector_async_hook.js',
'lib/internal/js_stream_socket.js',
'lib/internal/linkedlist.js',
Expand Down

0 comments on commit 0efdeea

Please sign in to comment.