Skip to content

Commit

Permalink
requireNative doesn't depend on rest of module system
Browse files Browse the repository at this point in the history
  • Loading branch information
ry committed Nov 21, 2010
1 parent 11b2ee7 commit bb6d468
Showing 1 changed file with 21 additions and 11 deletions.
32 changes: 21 additions & 11 deletions src/node.js
Original file line number Diff line number Diff line change
Expand Up @@ -104,22 +104,32 @@ var module = (function () {
// Like, natives.fs is the contents of lib/fs.js
var natives = process.binding('natives');

function loadNative (id) {
// Native modules don't need a full require function. So we can bootstrap
// most of the system with this mini-require.
function requireNative (id) {
if (internalModuleCache[id]) return internalModuleCache[id].exports;
if (!natives[id]) throw new Error('No such native module ' + id);

// REPL is a special case, because it needs the real require.
if (id == 'repl') {
var replModule = new Module("repl");
replModule._compile(natives.repl, 'repl.js');
internalModuleCache.repl = replModule;
return replModule.exports;
}

var fn = process.compile(
"(function (exports, require) {" + natives[id] + "\n})",
id + '.js');
var m = new Module(id);
fn(m.exports, requireNative);
m.loaded = true;
internalModuleCache[id] = m;
var e = m._compile(natives[id], id+".js");
if (e) throw e; // error compiling native module
return m;
return m.exports;
}

exports.requireNative = requireNative;

function requireNative (id) {
if (internalModuleCache[id]) return internalModuleCache[id].exports;
if (!natives[id]) throw new Error('No such native module ' + id);
return loadNative(id).exports;
}


// Modules

Expand Down Expand Up @@ -230,7 +240,7 @@ var module = (function () {
}
if (natives[id]) {
debug('load native module ' + request);
return loadNative(id).exports;
return requireNative(id);
}

var cachedModule = moduleCache[filename];
Expand Down

0 comments on commit bb6d468

Please sign in to comment.