fs: replace a bind() with a top-level function

#11225 introduce an unnecessary
bind() when closing a stream. This PR replaces that bind() with a
top-level function.

PR-URL: #13474
Reviewed-By: Refael Ackermann <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Brian White <[email protected]>
Reviewed-By: Evan Lucas <[email protected]>

Author: mcollina

lib/fs.js

@@ -2016,7 +2016,7 @@ ReadStream.prototype.close = function(cb) {
 
   if (this.closed || typeof this.fd !== 'number') {
     if (typeof this.fd !== 'number') {
-      this.once('open', this.close.bind(this, null));
+      this.once('open', closeOnOpen);
       return;
     }
     return process.nextTick(() => this.emit('close'));
@@ -2034,6 +2034,11 @@ ReadStream.prototype.close = function(cb) {
   this.fd = null;
 };
 
+// needed because as it will be called with arguments
+// that does not match this.close() signature
+function closeOnOpen(fd) {
+  this.close();
+}
 
 fs.createWriteStream = function(path, options) {
   return new WriteStream(path, options);