Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Introduce JS_ReadObject2 #468

Merged
merged 2 commits into from
Jul 15, 2024
Merged

Introduce JS_ReadObject2 #468

merged 2 commits into from
Jul 15, 2024

Conversation

saghul
Copy link
Contributor

@saghul saghul commented Jul 15, 2024

Analogously to JS_WriteObject2, it allows the user to get a tab with all the SAB objects that were read.

This can help adjust reference counts in a scenario where a SAB that was written increased it and it's necessary to decrease it upon reading it.

Analogously to JS_WriteObject2, it allows the user to get a tab with all
the SAB objects that were read.

This can help adjust reference counts in a scenario where a SAB that was
written increased it and it's necessary to decrease it upon reading it.
@saghul saghul requested a review from chqrlie July 15, 2024 07:25
@saghul
Copy link
Contributor Author

saghul commented Jul 15, 2024

@chqrlie PTAL. Context: my Worker implementation sends the serialized data over a socketpair, so being able to dup the SAB upon write and free it upon read would make my life easier :-)

@chqrlie
Copy link
Collaborator

chqrlie commented Jul 15, 2024

@chqrlie PTAL. Context: my Worker implementation sends the serialized data over a socketpair, so being able to dup the SAB upon write and free it upon read would make my life easier :-)

I have very little connectivity ATM, sorry for the lag.

@saghul
Copy link
Contributor Author

saghul commented Jul 15, 2024

@chqrlie PTAL. Context: my Worker implementation sends the serialized data over a socketpair, so being able to dup the SAB upon write and free it upon read would make my life easier :-)

I have very little connectivity ATM, sorry for the lag.

No worries, I just posted this today :-P

@saghul
Copy link
Contributor Author

saghul commented Jul 15, 2024

@chqrlie Updated, PTAL when you can!

@chqrlie
Copy link
Collaborator

chqrlie commented Jul 15, 2024

@chqrlie Updated, PTAL when you can!

@saghul LGTM

@saghul saghul merged commit 7630106 into master Jul 15, 2024
50 checks passed
@saghul saghul deleted the read2 branch July 15, 2024 14:53
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants