Skip to content

Bug: define() missing gpa argument in addClass call #197

@fecet

Description

@fecet

Summary

define() in src/define.zig calls addClass with 2 arguments, but the function requires 3 — the gpa: std.mem.Allocator parameter is missing.

Location

src/define.zig, line 36:

pub fn define(
    gpa: std.mem.Allocator,
    absolute_output_path: []const u8,
    comptime to_define: []const type,
) !void {
    var state: DefineState = .empty;
    defer state.deinit(gpa);

    inline for (to_define) |T| {
        _ = try addClass(&state, T);  // <- missing gpa
    }

addClass signature (line 88):

pub fn addClass(
    state: *DefineState,
    gpa: std.mem.Allocator,   // required
    comptime T: type,
) !void {

Fix

-        _ = try addClass(&state, T);
+        _ = try addClass(&state, gpa, T);

Version

Tag 0.6.0 (315b7f25efd29d9e2143d6c0ab685bb7f640450f)

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions