public/browser
1
0
Fork 0
mirror of https://github.com/lightpanda-io/browser.git synced 2025-10-29 07:03:29 +00:00
Code Issues Actions 4 Packages Projects Releases Wiki Activity

deinit persistent in dynamic import

Browse Source
This commit is contained in:
Muki Kiboigo
2025-09-09 10:54:02 -07:00
parent 2ed8a1c0ec
commit 7eb53ca2bc
1 changed files with 8 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/runtime/js.zig
View File

@@ -1668,6 +1668,11 @@ pub fn Env(comptime State: type, comptime WebApis: type) type {
specifier: []const u8, specifier: []const u8,
module: v8.Persistent(v8.Module), module: v8.Persistent(v8.Module),
resolver: v8.Persistent(v8.PromiseResolver), resolver: v8.Persistent(v8.PromiseResolver),
pub fn deinit(ev: *@This()) void {
ev.module.deinit();
ev.resolver.deinit();
}
}; };
const ev_data = try self.context_arena.create(EvaluationData); const ev_data = try self.context_arena.create(EvaluationData);
@@ -1684,6 +1689,7 @@ pub fn Env(comptime State: type, comptime WebApis: type) type {
const cb_isolate = cb_info.getIsolate(); const cb_isolate = cb_info.getIsolate();
const cb_context = cb_isolate.getCurrentContext(); const cb_context = cb_isolate.getCurrentContext();
const data: *EvaluationData = @ptrCast(@alignCast(cb_info.getExternalValue())); const data: *EvaluationData = @ptrCast(@alignCast(cb_info.getExternalValue()));
defer data.deinit();
const cb_module = data.module.castToModule(); const cb_module = data.module.castToModule();
const cb_resolver = data.resolver.castToPromiseResolver(); const cb_resolver = data.resolver.castToPromiseResolver();
@@ -1698,6 +1704,7 @@ pub fn Env(comptime State: type, comptime WebApis: type) type {
const cb_info = v8.FunctionCallbackInfo{ .handle = info.? }; const cb_info = v8.FunctionCallbackInfo{ .handle = info.? };
const cb_context = cb_info.getIsolate().getCurrentContext(); const cb_context = cb_info.getIsolate().getCurrentContext();
const data: *EvaluationData = @ptrCast(@alignCast(cb_info.getExternalValue())); const data: *EvaluationData = @ptrCast(@alignCast(cb_info.getExternalValue()));
defer data.deinit();
const cb_resolver = data.resolver.castToPromiseResolver(); const cb_resolver = data.resolver.castToPromiseResolver();
log.err(.js, "dynamic import failed", .{ .specifier = data.specifier }); log.err(.js, "dynamic import failed", .{ .specifier = data.specifier });
@@ -1710,6 +1717,7 @@ pub fn Env(comptime State: type, comptime WebApis: type) type {
.specifier = specifier, .specifier = specifier,
.line = try_catch.sourceLineNumber() orelse 0, .line = try_catch.sourceLineNumber() orelse 0,
}); });
defer ev_data.deinit();
const error_msg = v8.String.initUtf8(iso, "Evaluation is a promise"); const error_msg = v8.String.initUtf8(iso, "Evaluation is a promise");
_ = resolver.reject(ctx, error_msg.toValue()); _ = resolver.reject(ctx, error_msg.toValue());
return; return;