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.
  • Loading branch information
Trott committed Jun 19, 2021
1 parent 98139df commit ee6daab
Show file tree
Hide file tree
Showing 5 changed files with 6 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -39,8 +39,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
Original file line number Diff line number Diff line change
Expand Up @@ -13,5 +13,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
Original file line number Diff line number Diff line change
Expand Up @@ -143,6 +143,9 @@
'lib/internal/crypto/webcrypto.js',
'lib/internal/crypto/x509.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 @@ -166,9 +169,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/legacy/processbinding.js',
Expand Down

0 comments on commit ee6daab

Please sign in to comment.