Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/typed array support #90

Merged
merged 2 commits into from
Jun 5, 2019

Conversation

taxilian
Copy link
Contributor

It's much more efficient to use typed arrays when possible for things like audio -- this PR detects if typed arrays are available and uses them if they are. Since typed arrays have all the same features as normal arrays (and then some) this shouldn't break anything but for anyone who is using the advanced features (particularly if they are normalizing the audio) this will make a significant improvement, saving a lot of unneeded copying of data.

@edimuj
Copy link
Owner

edimuj commented Jun 5, 2019

Good work!

@edimuj edimuj merged commit 90e9034 into edimuj:master Jun 5, 2019
@sachingarg05
Copy link

Do we really need 'normalizeNoTyped' anymore? Do you have any information on which iOS/Android versions are likely to not have support for typed arrays?

@rumbcam
Copy link

rumbcam commented Jul 3, 2019

I see that this is merged, but I'm not seeing it as released to npm. Could we get a new release on npm that includes this?

@edimuj
Copy link
Owner

edimuj commented Jul 8, 2019

@rumbcam Sorry about that, the new release 1.0.2 has now been published on npm.

@donotlb
Copy link

donotlb commented Oct 31, 2019

⚠️ please take a look at this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants