Skip to content

module: rename anonymous functions #20021

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
ghost wants to merge 4 commits into from
Closed

module: rename anonymous functions #20021

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
15a616a
domain.js: Converted anonymous to named function
fmterrorf Apr 14, 2018
13a16e4
remove added names on prototypes
fmterrorf Apr 14, 2018
270b575
remove added names on prototypes
fmterrorf Apr 14, 2018
bfc18de
remove added names on prototypes
fmterrorf Apr 14, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
remove added names on prototypes
  • Loading branch information
@fmterrorf
fmterrorf committed Apr 14, 2018
commit 13a16e4fe44c2211f66601a06be0025d6bdf99ce
14 changes: 7 additions & 7 deletions lib/domain.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -266,7 +266,7 @@ Domain.prototype._errorHandler = function _errorHandler(er) {
};


Domain.prototype.enter = function enter() {
Domain.prototype.enter = function() {
// note that this might be a no-op, but we still need
// to push it onto the stack so that we can pop it later.
exports.active = process.domain = this;
Expand All @@ -275,7 +275,7 @@ Domain.prototype.enter = function enter() {
};


Domain.prototype.exit = function exit() {
Domain.prototype.exit = function() {
// don't do anything if this domain is not on the stack.
var index = stack.lastIndexOf(this);
if (index === -1) return;
Expand All @@ -290,7 +290,7 @@ Domain.prototype.exit = function exit() {


// note: this works for timers as well.
Domain.prototype.add = function add(ee) {
Domain.prototype.add = function(ee) {
// If the domain is already added, then nothing left to do.
if (ee.domain === this)
return;
Expand Down Expand Up @@ -319,15 +319,15 @@ Domain.prototype.add = function add(ee) {
};


Domain.prototype.remove = function remove(ee) {
Domain.prototype.remove = function(ee) {
ee.domain = null;
var index = this.members.indexOf(ee);
if (index !== -1)
this.members.splice(index, 1);
};


Domain.prototype.run = function run(fn) {
Domain.prototype.run = function(fn) {
var ret;

this.enter();
Expand Down Expand Up @@ -377,7 +377,7 @@ function intercepted(_this, self, cb, fnargs) {
}


Domain.prototype.intercept = function intercept(cb) {
Domain.prototype.intercept = function(cb) {
var self = this;

function runIntercepted() {
Expand All @@ -402,7 +402,7 @@ function bound(_this, self, cb, fnargs) {
}


Domain.prototype.bind = function bind(cb) {
Domain.prototype.bind = function(cb) {
var self = this;

function runBound() {
Expand Down