fuse-friends / fuse-native

Multithreaded FUSE bindings for Node JS.
203 stars 29 forks source link

fix: remove incorrect flush parameter #18

Closed piedar closed 4 years ago

piedar commented 4 years ago

I have some code like this

flush (path, fd, reply) {
  debug("flush", { path, fd })

  writer.flush(path)
    .then(() => reply(0))
    .catch(err => {
      debug({ err })
      reply(errorToFuseCode(err))
    })
}

but it produces

UnhandledPromiseRejectionWarning: TypeError: reply is not a function

because an extra datasync parameter gets in the way of the documented reply function.

This change removes the incorrect extra parameter.

mafintosh commented 4 years ago

Thanks