Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit 6ad458b

Browse filesBrowse files
committed
module: remove unnecessary property and method
`require.paths` property and `require.registerExtension` function have been throwing errors when used. They both are like this for years now. This patch removes them from the system. PR-URL: #2922 Reviewed-By: Roman Reiss <me@silverwind.io> Reviewed-By: Brian White <mscdex@mscdex.net> Reviewed-By: Сковорода Никита Андреевич <chalkerx@gmail.com> Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
1 parent 8853388 commit 6ad458b
Copy full SHA for 6ad458b

File tree

Expand file treeCollapse file tree

2 files changed

+0
-14
lines changed
Open diff view settings
Filter options
Expand file treeCollapse file tree

2 files changed

+0
-14
lines changed
Open diff view settings
Collapse file

‎lib/module.js‎

Copy file name to clipboardExpand all lines: lib/module.js
-10Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -389,20 +389,10 @@ Module.prototype._compile = function(content, filename) {
389389
return Module._resolveFilename(request, self);
390390
};
391391

392-
Object.defineProperty(require, 'paths', { get: function() {
393-
throw new Error('require.paths is removed. Use ' +
394-
'node_modules folders, or the NODE_PATH ' +
395-
'environment variable instead.');
396-
}});
397-
398392
require.main = process.mainModule;
399393

400394
// Enable support to add extra extension types
401395
require.extensions = Module._extensions;
402-
require.registerExtension = function() {
403-
throw new Error('require.registerExtension() removed. Use ' +
404-
'require.extensions instead.');
405-
};
406396

407397
require.cache = Module._cache;
408398

Collapse file

‎test/sequential/test-module-loading.js‎

Copy file name to clipboardExpand all lines: test/sequential/test-module-loading.js
-4Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -127,10 +127,6 @@ assert.equal(require('../fixtures/registerExt2').custom, 'passed');
127127
assert.equal(require('../fixtures/foo').foo, 'ok',
128128
'require module with no extension');
129129

130-
assert.throws(function() {
131-
require.paths;
132-
}, /removed/, 'Accessing require.paths should throw.');
133-
134130
// Should not attempt to load a directory
135131
try {
136132
require('../fixtures/empty');

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.