TooTallNate / node-speaker

Output PCM audio data to the speakers
655 stars 147 forks source link

make node-speaker node10x compatible, use internal stream node module #119

Closed hlolli closed 6 years ago

hlolli commented 6 years ago

This is actually no deadly necessary pull request. This mainly fixes bunch of deprecation warnings from Nan. I was in an attempt to fix a problem with new thread-workers in Node10 so I made these changes. But then found out that it wasn't the root of my problem https://github.com/nodejs/node/issues/21481 otherwise the tests run, produces expected audio on my computer, all deps are bumped, with the readable dep removed in favor of the internal stream module provided by nodejs ecosystem. The old readable-stream module is mostly a polyfill for browser compatability as far as I understood it, bumping it to v3x caused errors with node-speaker.