Skip to content

Commit 3e14449

Browse files
bengljasnell
authored andcommitted
fs: remove no-longer-relevant comment
The comment suggests that the subsequent code could by DRYed up, due to simply passing arguments along. However, in the commits since then, this no longer appears to apply, and so the comment is now confusing with respect to the current code. PR-URL: #16285 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Refael Ackermann <refack@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Evan Lucas <evanlucas@me.com> Reviewed-By: James M Snell <jasnell@gmail.com>
1 parent 880415e commit 3e14449

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

lib/fs.js

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -621,10 +621,6 @@ fs.readFileSync = function(path, options) {
621621
return buffer;
622622
};
623623

624-
625-
// Yes, the follow could be easily DRYed up but I provide the explicit
626-
// list to make the arguments clear.
627-
628624
fs.close = function(fd, callback) {
629625
var req = new FSReqWrap();
630626
req.oncomplete = makeCallback(callback);

0 commit comments

Comments
 (0)