Skip to content

bump to rquickjs 0.14 - #5

Merged
Sytten merged 4 commits into
rquickjs:mainfrom
svix-jbrown:chore/rquickjs-0.14
Oct 10, 2026
Merged

Sytten merged 4 commits into
rquickjs:mainfrom
svix-jbrown:chore/rquickjs-0.14

Conversation

@svix-jbrown

Copy link
Copy Markdown
Contributor

I am almost positive that the CFFI code here is unsound and results in UB because it transmutes a *mut u8 into a *const u8 while other references are live.

Every module except the sqlite one should be fine, though.

Part of #4

@Sytten

Sytten commented Oct 10, 2026 •

Copy link
Copy Markdown
Member

@svix-jbrown The blob was unsound but not because of the transmute mainly because the engine can mutate the pointer. We don't have a choice but to make a copy since we can yield to the engine in some cases.

I dropped the older versions and improved the general code while I was at it.

@Sytten
Sytten merged commit d972c8f into rquickjs:main Oct 10, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants