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 <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Refael Ackermann <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Evan Lucas <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent 880415e commit 3e14449

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

lib/fs.js

-4
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)