Skip to content

Commit 6a2b204

Browse files
Herbert Vojčíkcjihrig
Herbert Vojčík
authored andcommitted
module: replace NativeModule.require
The NativeModule system passes NativeModule.require transparently and so is unnecessary to call explicitly. The only one which should have the prefix is the in line 295, where actually implements a big fs-based module system and actually requires a native module. That is left unchanged. PR-URL: nodejs/node-v0.x-archive#9201 Ref: nodejs/node-v0.x-archive#2009 Reviewed-by: Trevor Norris <[email protected]> Conflicts: lib/module.js
1 parent 8c1df7a commit 6a2b204

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

lib/module.js

+3-5
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,12 @@
11
'use strict';
22

33
const NativeModule = require('native_module');
4-
const util = NativeModule.require('util');
4+
const util = require('util');
55
const runInThisContext = require('vm').runInThisContext;
66
const runInNewContext = require('vm').runInNewContext;
77
const assert = require('assert').ok;
8-
const fs = NativeModule.require('fs');
8+
const fs = require('fs');
9+
const path = require('path');
910

1011

1112
// If obj.hasOwnProperty has been overridden, then calling
@@ -41,9 +42,6 @@ Module.globalPaths = [];
4142

4243
Module.wrapper = NativeModule.wrapper;
4344
Module.wrap = NativeModule.wrap;
44-
45-
const path = NativeModule.require('path');
46-
4745
Module._debug = util.debuglog('module');
4846

4947

0 commit comments

Comments
 (0)