Permalink
Browse files

core: make .deprecate() warn only once

  • Loading branch information...
1 parent 5403a8c commit 52bd0f93bb786de028463ef56ddb1e22568584b9 Brandon Benvie committed with bnoordhuis Jan 30, 2012
Showing with 15 additions and 6 deletions.
  1. +15 −6 src/node.js
View
@@ -569,21 +569,30 @@
NativeModule._cache[this.id] = this;
};
+ // Wrap a core module's method in a wrapper that will warn on first use
+ // and then return the result of invoking the original function. After
+ // first being called the original method is restored.
NativeModule.prototype.deprecate = function(method, message) {
var original = this.exports[method];
var self = this;
+ var warned = false;
+ message = message || '';
Object.defineProperty(this.exports, method, {
enumerable: false,
value: function() {
- message = self.id + '.' + method + ' is deprecated. ' + (message || '');
+ if (!warned) {
+ warned = true;
+ message = self.id + '.' + method + ' is deprecated. ' + message;
- if ((new RegExp('\\b' + self.id + '\\b')).test(process.env.NODE_DEBUG))
- console.trace(message);
- else
- console.error(message);
+ var moduleIdCheck = new RegExp('\\b' + self.id + '\\b');
+ if (moduleIdCheck.test(process.env.NODE_DEBUG))
+ console.trace(message);
+ else
+ console.error(message);
- self.exports[method] = original;
+ self.exports[method] = original;
+ }
return original.apply(this, arguments);
}
});

0 comments on commit 52bd0f9

Please sign in to comment.