Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

Commit

Permalink
Browse files Browse the repository at this point in the history
process: remove old notices about removed methods
These methods were removed a long time ago. Keeping these notices here
makes no sense anymore.

Also, removing this part of code slightly speeds up the startup.
  • Loading branch information
mmalecki authored and isaacs committed Feb 16, 2012
1 parent 71e9756 commit 3bc3af0
Showing 1 changed file with 0 additions and 33 deletions.
33 changes: 0 additions & 33 deletions src/node.js
Expand Up @@ -46,8 +46,6 @@

startup.processChannel();

startup.removedMethods();

startup.resolveArgv0();

// There are various modes that Node can run in. The most common two
Expand Down Expand Up @@ -431,37 +429,6 @@
}
}

startup._removedProcessMethods = {
'assert': 'process.assert() use require("assert").ok() instead',
'debug': 'process.debug() use console.error() instead',
'error': 'process.error() use console.error() instead',
'watchFile': 'process.watchFile() has moved to fs.watchFile()',
'unwatchFile': 'process.unwatchFile() has moved to fs.unwatchFile()',
'mixin': 'process.mixin() has been removed.',
'createChildProcess': 'childProcess API has changed. See doc/api.txt.',
'inherits': 'process.inherits() has moved to util.inherits()',
'_byteLength': 'process._byteLength() has moved to Buffer.byteLength'
};

startup.removedMethods = function() {
var desc = {
configurable: true,
writable: true,
enumerable: false
};
for (var method in startup._removedProcessMethods) {
var reason = startup._removedProcessMethods[method];
desc.value = startup._removedMethod(reason);
Object.defineProperty(process, method, desc);
}
};

startup._removedMethod = function(reason) {
return function() {
throw new Error(reason);
};
};

startup.resolveArgv0 = function() {
var cwd = process.cwd();
var isWindows = process.platform === 'win32';
Expand Down

0 comments on commit 3bc3af0

Please sign in to comment.