mozjs icon indicating copy to clipboard operation
mozjs copied to clipboard

Added Glue for Functions with `mozilla::Range` Parameters

Open Redfire75369 opened this issue 1 year ago • 2 comments

Small change to #433. slice::as_mut_ptr_range gives a Range<*mut T>, which would need to be converted to a reference to be used.

Redfire75369 avatar Dec 02 '23 15:12 Redfire75369

Would it be possible to also add test for Range?

sagudev avatar Dec 02 '23 15:12 sagudev

The errors seem to be caused by a C++ ABI issue, which is annoying, as per usual. The moment the Range<T> passes to C++, all hell breaks loose and the pointer fields point to nonsense.

Redfire75369 avatar Dec 02 '23 17:12 Redfire75369