DelSkayn / rquickjs

High level bindings to the quickjs javascript engine
MIT License
504 stars 63 forks source link

Invalid allocation alligment on `msvc` #157

Closed DelSkayn closed 1 year ago

DelSkayn commented 1 year ago

Currently, when implementing a custom allocator for quickjs the library makes sure that allocations are 64 bit aligned on a 64 bit target and 32 bit aligned on a 32 bit target. However, the msvc doesn't follow this rule, instead 64bit values are 64 bit aligned even on 32 bit target.

We should change to implementation to reflect this.