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.

Sign up for GitHub

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

Jump to bottom

translate-c "function parameter has no name" #4030

Closed
daurnimator opened this issue Jan 2, 2020 · 0 comments · Fixed by #4033
Closed

translate-c "function parameter has no name" #4030

daurnimator opened this issue Jan 2, 2020 · 0 comments · Fixed by #4033
Labels
contributor friendly This issue is limited in scope and/or knowledge of Zig internals. frontend Tokenization, parsing, AstGen, Sema, and Liveness. translate-c C to Zig source translation feature (@cImport)
Milestone
0.6.0

Comments

@daurnimator
Copy link
Contributor

e.g. from #2314

static void foo() {}

Currently gets translated to:

pub const foo = @compileError("function foo parameter has no name");

translate-c could just make up a name for an unnamed parameter.

Possibly related article/comments:
@daurnimator daurnimator added the translate-c C to Zig source translation feature (@cImport) label Jan 2, 2020
@Vexu Vexu mentioned this issue Jan 2, 2020
Merged
@andrewrk andrewrk added this to the 0.7.0 milestone Jan 2, 2020
@andrewrk andrewrk added contributor friendly This issue is limited in scope and/or knowledge of Zig internals. frontend Tokenization, parsing, AstGen, Sema, and Liveness. labels Jan 2, 2020
@andrewrk andrewrk modified the milestones: 0.7.0, 0.6.0 Feb 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
contributor friendly This issue is limited in scope and/or knowledge of Zig internals. frontend Tokenization, parsing, AstGen, Sema, and Liveness. translate-c C to Zig source translation feature (@cImport)
Projects
None yet
Milestone
0.6.0
Development

Successfully merging a pull request may close this issue.

translate-c properly handle unused var-args Vexu/zig
2 participants
@andrewrk @daurnimator