28 ms·
True in some cases I fold back to JSON: https://github.com/mcuadros/go-candyjs/blob/master/base.go#L272 https://github.com/mcuadros/go-candyjs/blob/master/base.
by mcuadros 11y ago
True in some cases I fold back to JSON:
https://github.com/mcuadros/go-candyjs/blob/master/base.go#L272 https://github.com/mcuadros/go-candyjs/blob/master/base.go#L...
But the main goal of the project is be fully transparent more than the performance. At the very beginning I was making every case by hand but this is a endless work:
https://github.com/mcuadros/go-candyjs/blob/54c8beb723aa8b1b21f2d5864b0809f326d9a85d/engine/duktape/register.go#L142-L167 https://github.com/mcuadros/go-candyjs/blob/54c8beb723aa8b1b...
I will take a closer look to the NaN issue and also a closer look to your code.
BTW I made a PR to go-duktape based on you fork:
https://github.com/olebedev/go-duktape/commit/65f0be48ece4f6405c93c3b5312fab074320bfef https://github.com/olebedev/go-duktape/commit/65f0be48ece4f6...
- ivan4th 11y agoUnfortunately NaN behavior seems to be hard-wired in encoding/json code. The proper way to fix it is perhaps taking encoding/json sources and converting marshalling/unmarshalling code to use go-duktape Push/To/etc. functions instead of JSON writing/parsing. That's indeed a sizable amount of work, though. As of integrating (some of) changes from my fork to the go-duktape mainline, thanks a lot! Didn't get around to do it myself.