zig/mach
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

module: improve error messages when systems are registered incorrectly

Browse source 
Signed-off-by: Emi Gutekanst <emi@hexops.com>
This commit is contained in:
Emi Gutekanst 2025-01-11 17:12:57 -07:00
parent eb12b0f476
commit 1e2cbc4d71
1 changed files with 6 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

6
src/module.zig
View file

@ -609,6 +609,9 @@ pub fn Modules(module_lists: anytype) type {
comptime fn_name: ModuleFunctionName(module_name), comptime fn_name: ModuleFunctionName(module_name),
) void { ) void {
const debug_name = @tagName(module_name) ++ "." ++ @tagName(fn_name); const debug_name = @tagName(module_name) ++ "." ++ @tagName(fn_name);
if (!@hasField(module, @tagName(fn_name)) and !@hasDecl(module, @tagName(fn_name))) {
@compileError("Module ." ++ @tagName(module_name) ++ " declares mach_systems entry ." ++ @tagName(fn_name) ++ " but no pub fn or schedule with that name exists.");
}
const f = @field(module, @tagName(fn_name)); const f = @field(module, @tagName(fn_name));
const F = @TypeOf(f); const F = @TypeOf(f);
@ -619,6 +622,9 @@ pub fn Modules(module_lists: anytype) type {
inline for (f) |schedule_entry| { inline for (f) |schedule_entry| {
// TODO: unify with Modules(modules).get(M) // TODO: unify with Modules(modules).get(M)
const callMod: Module(schedule_entry.@"0") = .{ .mods = m.mods, .modules = m.modules }; const callMod: Module(schedule_entry.@"0") = .{ .mods = m.mods, .modules = m.modules };
if (!@hasField(ModuleFunctionName(@TypeOf(callMod).mod_name), @tagName(schedule_entry.@"1"))) {
@compileError("Module ." ++ @tagName(@TypeOf(callMod).mod_name) ++ " has no mach_systems entry '." ++ @tagName(schedule_entry.@"1") ++ "'");
}
const callFn = @as(ModuleFunctionName(@TypeOf(callMod).mod_name), schedule_entry.@"1"); const callFn = @as(ModuleFunctionName(@TypeOf(callMod).mod_name), schedule_entry.@"1");
callMod.run(callFn); callMod.run(callFn);
} }