-
Notifications
You must be signed in to change notification settings - Fork 56
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
Implement Error.prepareStackTrace support #223
Conversation
@bnoordhuis Can you PTAL? It's not fully finished yet (see TODOs) but I'd like some feedback on the approach. The tricky part was to avoid doing anything while iterating through the stack frames since they could get modified while we are iterating over them. Edit: also, do you reckon there is more low hanging fruit I could add, to make the API closer to V8's? |
da9c7b2
to
66c95b4
Compare
Alright, this is one is ready now, PTAL @bnoordhuis ! |
Based on V8's API: https://v8.dev/docs/stack-trace-api. Bits picked from Frida: frida/quickjs@78fd25f Closes: #134
66c95b4
to
07fad07
Compare
🔥 |
Based on V8's API: https://v8.dev/docs/stack-trace-api.
Bits picked from Frida: frida/quickjs@78fd25f
Closes: #134