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

Issue 8848 - Array literals and AA literals are rejected as template value parameters #2871

Merged
1 commit merged into from Feb 8, 2014

Conversation

yebblies
Copy link
Member

The check is unnecessary as pretty much everything can be passed, and expressions that can't will be detected when they are interpreted.

https://d.puremagic.com/issues/show_bug.cgi?id=8848

@yebblies
Copy link
Member Author

Looking pretty green!

@ghost
Copy link

ghost commented Dec 19, 2013

Nice. This will be worthy of a changelog entry later.

@@ -2508,6 +2508,10 @@ void test9977()
}

/******************************************/

template T8848(int[] a, int[int] b, void* c) {}

Copy link
Member

Choose a reason for hiding this comment

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

Needs an attempt to instantiate the template, and use the instantiation to verify it is working as expected. Parsing it successfully is not enough.

Copy link
Member Author

Choose a reason for hiding this comment

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

Ok, although it was enough for the fail_compilation test.

Copy link

Choose a reason for hiding this comment

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

Ping for test-cases. :)

@yebblies
Copy link
Member Author

yebblies commented Feb 8, 2014

Added some tests.

@ghost
Copy link

ghost commented Feb 8, 2014

Auto-merge toggled on

ghost pushed a commit that referenced this pull request Feb 8, 2014
Issue 8848 - Array literals and AA literals are rejected as template value parameters
@ghost ghost merged commit ae5e8fa into dlang:master Feb 8, 2014
@yebblies yebblies deleted the issue8488 branch February 8, 2014 23:50
@yebblies
Copy link
Member Author

yebblies commented Feb 8, 2014

Thanks!

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants