-
-
Notifications
You must be signed in to change notification settings - Fork 740
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
[ESP8266] new Uint8Array -> reset #2426
Comments
This works in the normal build, so what hardware are you running this on? And is it a build we supply or one you did yourself? |
This is running on an ESP8266 with firmware downloaded from the releases page (here to be specific) I tried it on a second chip just to rule out bad hardware. But I get the same issue there. |
Also tried on a completely fresh install, no JS loaded whatsoever. Just cli using Espruino binary from NPM |
Ok, thanks!
Are you sure? Because it says |
Yes, initial bug report was indeed on a chip that had other JS loaded on it. Here's the output after running
|
Ok, thanks - it's interesting this one lacks the |
Another interesting observation; it will allows me to create signed integers once, but if I do it twice then it will reset. Reproducible every time.
|
This appears to be fixed now - I'm pretty sure it was due to an overflow in the JsVar union for arraybuffers, which only showed up on devices with little RAM |
Great, thanks! I can confirm. Tested my old code on v2.20 and it's running fine now. |
Previously I was on version 2.14 and all was fine. But with 2.18 and 2.19 I have this error:
The text was updated successfully, but these errors were encountered: