Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The @"" issue #5

Open
devins2518 opened this issue Oct 27, 2022 · 0 comments
Open

The @"" issue #5

devins2518 opened this issue Oct 27, 2022 · 0 comments

Comments

@devins2518
Copy link
Owner

devins2518 commented Oct 27, 2022

RIP @"".

A quick bandaid was to just change relevant enums to have a none tag, however, changing them to optional enums is a valid alternative. AFAIK zig does not optimize the following to the same size just yet.

const A = enum {
    a,
    b,
    none,
};
const B = ?enum {
    a,
    b,
}

This approach could make formatting a bit nicer too, instead of needing an explicit if, we could just have a default value if it's null if that's possible through std.fmt.format.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant