ccxvii / mujs

An embeddable Javascript interpreter in C.
http://mujs.com/
ISC License
812 stars 98 forks source link

Get rid of setjmp #170

Closed trufae closed 1 year ago

trufae commented 1 year ago

Despite being a useful function and simplifies a lot the error handling it reduces portability and performance and causes issues when combined with threads, qjs is the only js runtime out there that it's not using it (even lua or duktape use setjmp 😱) but i find mujs super clean and simple and i wished about this, any thoughs on this?

ccxvii commented 1 year ago

It won't happen without a complete rewrite. The use of setjmp for error handling (and garbage collection for cleanup) is very pervasive throughout the code.