Page MenuHomePhabricator

[CommRustModule] improve params capturing
ClosedPublic

Authored by kamil on Jan 5 2024, 2:22 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Jan 21, 2:38 PM
Unknown Object (File)
Jan 10 2025, 6:52 AM
Unknown Object (File)
Jan 9 2025, 10:51 AM
Unknown Object (File)
Dec 29 2024, 8:25 AM
Unknown Object (File)
Dec 28 2024, 9:13 AM
Unknown Object (File)
Dec 23 2024, 4:24 AM
Unknown Object (File)
Dec 23 2024, 4:24 AM
Unknown Object (File)
Dec 23 2024, 4:24 AM
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