Skip to content

lib: avoid using Array.prototype.forEach #11582

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
jasnell wants to merge 11 commits into from
Closed

lib: avoid using Array.prototype.forEach #11582

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
b2c00ce
benchmark: benchmark comparing forEach with for
jasnell Feb 26, 2017
4d0d400
stream: avoid using forEach
jasnell Feb 26, 2017
a2d8baa
tls: avoid using forEach
jasnell Feb 26, 2017
ebe8926
fs: avoid using forEach
jasnell Feb 26, 2017
36dc38a
lib: avoid using forEach in LazyTransform
jasnell Feb 27, 2017
abdd6c6
net: avoid using forEach
jasnell Feb 27, 2017
e8738ca
readline: avoid using forEach
jasnell Feb 27, 2017
59d1fd6
repl: avoid using forEach
jasnell Feb 27, 2017
e9e5c0e
module: avoid using forEach
jasnell Feb 27, 2017
2e0cb91
util: avoid using forEach
jasnell Feb 27, 2017
142bf54
lib: avoid using forEach
jasnell Feb 27, 2017
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
fs: avoid using forEach
  • Loading branch information
@jasnell
jasnell committed Mar 15, 2017
commit ebe8926025601fccfec9798b581f7f89e53a3526
9 changes: 5 additions & 4 deletions lib/fs.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -248,10 +248,11 @@ function statsFromValues() {
}

// Don't allow mode to accidentally be overwritten.
['F_OK', 'R_OK', 'W_OK', 'X_OK'].forEach(function(key) {
Object.defineProperty(fs, key, {
enumerable: true, value: constants[key] || 0, writable: false
});
Object.defineProperties(fs, {
F_OK: {enumerable: true, value: constants.F_OK || 0},
R_OK: {enumerable: true, value: constants.R_OK || 0},
W_OK: {enumerable: true, value: constants.W_OK || 0},
X_OK: {enumerable: true, value: constants.X_OK || 0},
});

function handleError(val, callback) {
Expand Down