purescript-deprecated/purescript-node-streams-aff

callback foreign function

Closed this issue 3 years ago · 1 comments

jamesdbrock commented 3 years ago

https://github.com/jamesdbrock/purescript-node-streams-aff/blob/96810fdfcb4b2a8e8adf1e11b7ebb7a773953838/src/Node/Stream/Internal.js#L17

Should this be?

 s.once('error', (error) => f(error)());

jamesdbrock commented 3 years ago

Yes definitely.