Skip to content

Fix Issue #22790: Allow static array parameters in ImportC #22800

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

Open
oCHIKIo wants to merge 3 commits into
base: master
Choose a base branch
from
+28 −6
Open

Fix Issue #22790: Allow static array parameters in ImportC #22800

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
e52a52d
Fix Issue #22790: Allow static array parameters in ImportC
Mar 23, 2026
2e113d6
Add runnable test for valid static array parameters
Mar 24, 2026
8bb2341
Fix #22790 reviews: Add expressions and qualifiers static array param…
Apr 13, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 0 additions & 4 deletions compiler/src/dmd/cparse.d
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2994,10 +2994,6 @@ final class CParser(AST) : Parser!AST
{
error("variable length arrays are not supported");
}
if (isStatic) // C11 6.7.6.3
{
error("static array parameters are not supported");
}
if (declarator != DTR.xparameter)
{
/* C11 6.7.6.2-4: '*' can only be used with function prototype scope.
Expand Down
2 changes: 0 additions & 2 deletions compiler/test/fail_compilation/failcstuff1.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -32,14 +32,12 @@ fail_compilation/failcstuff1.c(260): Error: expected identifier for declarator
fail_compilation/failcstuff1.c(301): Error: illegal type combination
fail_compilation/failcstuff1.c(352): Error: found `2` when expecting `:`
fail_compilation/failcstuff1.c(352): Error: found `:` instead of statement
fail_compilation/failcstuff1.c(450): Error: static array parameters are not supported
fail_compilation/failcstuff1.c(450): Error: static or type qualifier used in non-outermost array type derivation
fail_compilation/failcstuff1.c(451): Error: static or type qualifier used in non-outermost array type derivation
fail_compilation/failcstuff1.c(451): Error: array type has incomplete element type `int[0]`
fail_compilation/failcstuff1.c(452): Error: array type has incomplete element type `int[0]`
fail_compilation/failcstuff1.c(453): Error: array type has incomplete element type `int[0]`
fail_compilation/failcstuff1.c(454): Error: found `const` when expecting `,`
fail_compilation/failcstuff1.c(458): Error: static array parameters are not supported
fail_compilation/failcstuff1.c(458): Error: static or type qualifier used outside of function prototype
fail_compilation/failcstuff1.c(459): Error: static or type qualifier used outside of function prototype
fail_compilation/failcstuff1.c(460): Error: variable length arrays are not supported
Expand Down
12 changes: 12 additions & 0 deletions compiler/test/runnable/test22790.c
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
// https://issues.dlang.org/show_bug.cgi?id=22790
Copy link
Copy Markdown
Contributor

@dkorpel dkorpel Apr 9, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be a GitHub link


void test_static(int array[static 4]) {
Copy link
Copy Markdown
Contributor

@thewilsonator thewilsonator Mar 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is the D type of this function?
void function(int*) or void function(int[4]*)?

_Static_assert(sizeof(array) == sizeof(int*), "array must decay to pointer");
}

int main()
{
int a[4];
test_static(a);
return 0;
Copy link
Copy Markdown
Contributor

@dkorpel dkorpel Apr 13, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This test is not asserting anything at run time, can you remove main and make it compilable? Runnable tests are very slow.

}
Loading