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

Updated nuklear to work on GNU compiler. #97

Merged
merged 1 commit into from
Mar 9, 2025

Conversation

aperfilev
Copy link

On GNU GDC compiler impossible to compile nuklear extension due to small problem:

While for DMD & LDC2 this is not an issue but in GDC
Variadic static array args cannot be passed to extern(C) method as value. Need to be converted into a pointer.

~/dagon/extensions/nuklear$ dub build --compiler=gdc
    Starting Performing "debug" build using gdc for x86_64.
    Building nuklear ~master: building configuration [library]
src/dagon/ext/nuklear.d:1096:56: error: cannot pass static arrays to ‘extern(C)’ vararg functions
 1096 |         nk_labelf_colored(&ctx, align_, color, format, args);
      |                                                        ^
src/dagon/ext/nuklear.d:1103:38: error: cannot pass static arrays to ‘extern(C)’ vararg functions
 1103 |         nk_labelf_wrap(&ctx, format, args);
      |                                      ^
src/dagon/ext/nuklear.d:1110:53: error: cannot pass static arrays to ‘extern(C)’ vararg functions
 1110 |         nk_labelf_colored_wrap(&ctx, color, format, args);
      |                                                     ^
Error gdc failed with exit code 1.

@aperfilev
Copy link
Author

I think pipeline failed due to network issue within pipeline

@aperfilev
Copy link
Author

@gecko0307 Please check these pipelines - they seems not updating dmd version correct.

@gecko0307 gecko0307 merged commit eb09e99 into gecko0307:master Mar 9, 2025
4 checks passed
@aperfilev aperfilev deleted the gnu_nuklear branch March 12, 2025 07:05
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

Successfully merging this pull request may close these issues.

2 participants