Page MenuHomePhabricator

[CommRustModule] improve params capturing
ClosedPublic

Authored by kamil on Jan 5 2024, 2:22 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 29, 8:25 AM
Unknown Object (File)
Sat, Dec 28, 9:13 AM
Unknown Object (File)
Mon, Dec 23, 4:24 AM
Unknown Object (File)
Mon, Dec 23, 4:24 AM
Unknown Object (File)
Mon, Dec 23, 4:24 AM
Unknown Object (File)
Mon, Dec 23, 4:23 AM
Unknown Object (File)
Thu, Dec 12, 1:31 AM
Unknown Object (File)
Nov 26 2024, 8:52 PM
Subscribers

Details

Summary

Addresses https://phab.comm.dev/D10510#inline-64080.

Alternative is using std::shared_ptr, but I don't think is safe because as far as I know jsi::* objects lifetime is managed by the JavaScript runtime, is tied to the JavaScript garbage collector, not by C++ here itself, so I am afraid it could potentially lead to dangling pointers.

Depends on D10530

Test Plan

Test entire stack (it uses methods here).

Diff Detail

Repository
rCOMM Comm
Branch
publish-olm-stack-3
Lint
No Lint Coverage
Unit
No Test Coverage