Skip to content

Commit f501062

Browse files
bengltargos
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 a601596 commit f501062

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
@@ -605,10 +605,6 @@ fs.readFileSync = function(path, options) {
605605
return buffer;
606606
};
607607

608-
609-
// Yes, the follow could be easily DRYed up but I provide the explicit
610-
// list to make the arguments clear.
611-
612608
fs.close = function(fd, callback) {
613609
var req = new FSReqWrap();
614610
req.oncomplete = makeCallback(callback);

0 commit comments

Comments
 (0)