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 fails to translate function declarations where name is in parenthesis #4289

Closed
daurnimator opened this issue Jan 26, 2020 · 0 comments · Fixed by #4316
Closed

translate-c fails to translate function declarations where name is in parenthesis #4289

daurnimator opened this issue Jan 26, 2020 · 0 comments · Fixed by #4316
Labels
bug Observed behavior contradicts documented or intended behavior 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
Collaborator 

void (lua_close) (void *L);

gets translated to:

pub const lua_close = @compileError("unable to resolve function type ZigClangTypeClass.Paren");

This has regressed since 0.5.0.
@daurnimator daurnimator added bug Observed behavior contradicts documented or intended behavior translate-c C to Zig source translation feature (@cImport) labels Jan 26, 2020
@andrewrk andrewrk added this to the 0.6.0 milestone Jan 28, 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 28, 2020
LemonBoy added a commit to LemonBoy/zig that referenced this issue Jan 29, 2020
@LemonBoy
translate-c: Handle fn protos wrapped in parenthesis
6940121 
Closes ziglang#4289
@LemonBoy LemonBoy mentioned this issue Jan 29, 2020
Merged
LemonBoy added a commit to LemonBoy/zig that referenced this issue Jan 29, 2020
@LemonBoy
translate-c: Handle fn protos wrapped in parenthesis
20bc35b 
Closes ziglang#4289
andrewrk pushed a commit that referenced this issue Jan 29, 2020
@LemonBoy @andrewrk
translate-c: Handle fn protos wrapped in parenthesis
534014f 
Closes #4289
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Observed behavior contradicts documented or intended behavior 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: Handle fn protos wrapped in parenthesis LemonBoy/zig
2 participants
@andrewrk @daurnimator