node-tmp icon indicating copy to clipboard operation
node-tmp copied to clipboard

Support calling remove callback multiple times

Open nwalters512 opened this issue 1 year ago • 0 comments

While refactoring some code, I ended up accidentally calling the remove callback multiple times. It took me way too long to figure out why it seemed like my process was hanging: because the callback passed to the remove callback isn't invoked when the remove callback is called multiple times.

I acknowledge that one shouldn't end up calling the remove callback multiple times in normal usage, but it's a big violation of expectations for my own callback to never be called in that situation. I'm hoping you'll accept this patch. Thank you!

nwalters512 avatar Jun 10 '24 23:06 nwalters512