eliott/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird synced 2026-04-28 18:47:15 +02:00
Code Issues Packages Projects Releases Wiki Activity

LibJS: Generate FFI header using cbindgen instead of hand-rolling

Browse Source 
Replace the BytecodeFactory header with cbindgen.

This will help ensure that types and enums and constants are kept in
sync between the C++ and Rust code. It's also a step in exporting more
Rust enums directly rather than relying on magic constants for
switch statements.

The FFI functions are now all placed in the JS::FFI namespace, which
is the cause for all the churn in the scripting parts of LibJS and
LibWeb.
This commit is contained in:
Andrew Kaster
2026-03-17 15:47:58 -06:00
committed by Andreas Kling
parent c381cd7d68
commit 92e4c20ad5
Notes: github-actions[bot] 2026-03-18 01:50:46 +00:00 
Author: https://github.com/ADKaster
Commit: https://github.com/LadybirdBrowser/ladybird/commit/92e4c20ad5b
Pull-request: https://github.com/LadybirdBrowser/ladybird/pull/8476
19 changed files with 1167 additions and 557 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
Libraries/LibJS/SourceTextModule.cpp
View File

@@ -204,7 +204,7 @@ void SourceTextModule::visit_edges(Cell::Visitor& visitor)
visitor.visit(m_tla_shared_data);
}
Result<GC::Ref<SourceTextModule>, Vector<ParserError>> SourceTextModule::parse_from_pre_parsed(RustParsedProgram* parsed, NonnullRefPtr<SourceCode const> source_code, Realm& realm, Script::HostDefined* host_defined)
Result<GC::Ref<SourceTextModule>, Vector<ParserError>> SourceTextModule::parse_from_pre_parsed(FFI::ParsedProgram* parsed, NonnullRefPtr<SourceCode const> source_code, Realm& realm, Script::HostDefined* host_defined)
{
auto filename = source_code->filename();
auto rust_result = RustIntegration::compile_parsed_module(parsed, move(source_code), realm);