diff --git a/lib/fs.js b/lib/fs.js index 1c811ad8332009..194d609a4a0756 100644 --- a/lib/fs.js +++ b/lib/fs.js @@ -530,7 +530,7 @@ function close(fd, callback = defaultCloseCallback) { function closeSync(fd) { fd = getValidatedFd(fd); - return binding.close(fd); + binding.close(fd); } /** @@ -782,7 +782,7 @@ function readv(fd, buffers, position, callback) { if (typeof position !== 'number') position = null; - return binding.readBuffers(fd, buffers, position, req); + binding.readBuffers(fd, buffers, position, req); } ObjectDefineProperty(readv, kCustomPromisifyArgsSymbol, @@ -858,7 +858,8 @@ function write(fd, buffer, offsetOrOptions, length, position, callback) { const req = new FSReqCallback(); req.oncomplete = wrapper; - return binding.writeBuffer(fd, buffer, offset, length, position, req); + binding.writeBuffer(fd, buffer, offset, length, position, req); + return; } validateStringAfterArrayBufferView(buffer, 'buffer'); @@ -879,7 +880,7 @@ function write(fd, buffer, offsetOrOptions, length, position, callback) { const req = new FSReqCallback(); req.oncomplete = wrapper; - return binding.writeString(fd, str, offset, length, req); + binding.writeString(fd, str, offset, length, req); } ObjectDefineProperty(write, kCustomPromisifyArgsSymbol, @@ -969,7 +970,7 @@ function writev(fd, buffers, position, callback) { if (typeof position !== 'number') position = null; - return binding.writeBuffers(fd, buffers, position, req); + binding.writeBuffers(fd, buffers, position, req); } ObjectDefineProperty(writev, kCustomPromisifyArgsSymbol, { @@ -1175,7 +1176,8 @@ function rmdir(path, options, callback) { if (err === false) { const req = new FSReqCallback(); req.oncomplete = callback; - return binding.rmdir(path, req); + binding.rmdir(path, req); + return; } if (err) { return callback(err); @@ -1188,7 +1190,7 @@ function rmdir(path, options, callback) { validateRmdirOptions(options); const req = new FSReqCallback(); req.oncomplete = callback; - return binding.rmdir(path, req); + binding.rmdir(path, req); } } @@ -1294,7 +1296,7 @@ function fdatasync(fd, callback) { */ function fdatasyncSync(fd) { fd = getValidatedFd(fd); - return binding.fdatasync(fd); + binding.fdatasync(fd); } /** @@ -1319,7 +1321,7 @@ function fsync(fd, callback) { */ function fsyncSync(fd) { fd = getValidatedFd(fd); - return binding.fsync(fd); + binding.fsync(fd); } /** @@ -1868,7 +1870,7 @@ function unlink(path, callback) { */ function unlinkSync(path) { path = pathModule.toNamespacedPath(getValidatedPath(path)); - return binding.unlink(path); + binding.unlink(path); } /** @@ -2909,7 +2911,7 @@ realpath.native = (path, options, callback) => { path = getValidatedPath(path); const req = new FSReqCallback(); req.oncomplete = callback; - return binding.realpath(pathModule.toNamespacedPath(path), options.encoding, req); + binding.realpath(pathModule.toNamespacedPath(path), options.encoding, req); }; /**