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

Mp3 convert of a long record may results in a buffer over flow #23

Open
khelkun opened this issue Feb 5, 2016 · 3 comments
Open

Mp3 convert of a long record may results in a buffer over flow #23

khelkun opened this issue Feb 5, 2016 · 3 comments

Comments

@khelkun
Copy link

khelkun commented Feb 5, 2016

I've tried to record a 10 minutes sample on the demo page http://audior.ec/recordmp3js/
I click on stop button then it starts to convert to Mp3 and it raises a buffer over flow error after a few minutes

Converting to Mp3
Version : 3.99.5 /  Mode: 3 /  Samples: -1 /  Channels: 1 /  Input Samplate: 44100 /  Output Samplate: 44100 /  Bitlate :32 /  VBR :0
Uncaught buffer over flow
    Lame.encode_buffer_ieee_float @ /recordmp3js/js/libmp3lame.min.js:1446
    self.onmessage @ /recordmp3js/js/mp3Worker.js:31
@geebone
Copy link

geebone commented Mar 25, 2016

I'm getting this too

@kimlage
Copy link

kimlage commented Apr 10, 2017

same here... any ideas?
The problem here starts after 5 min records...
I could save a 4 min one....

@khelkun
Copy link
Author

khelkun commented Apr 13, 2017

I don't need Recordmp3js at present and no time to figure out the root cause of the issue but to do so I would: run the sample on my own environment with the non minified libmp3lame.js to see where the exception is raised in libmp3lame.js.

Then one should see if it's something related to libmp3lame.js itself (has not evolved for 4 years) or the usage done by Recordmp3js (has not evolved for 2 years) or the Recordmp3js sample code.
It could be related to this issue: akrennmair/libmp3lame-js#8
Then may be changing the Lame config would do the trick.

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

No branches or pull requests

3 participants