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 12378 - Compiler accepts any syntactically-valid code inside double-nested map predicate #3383

Merged
merged 3 commits into from Mar 23, 2014

Conversation

9rnsr
Copy link
Contributor

@9rnsr 9rnsr commented Mar 17, 2014

@WalterBright
Copy link
Member

Auto-merge toggled on

WalterBright added a commit that referenced this pull request Mar 23, 2014
Issue 12378 - Compiler accepts any syntactically-valid code inside double-nested map predicate
@WalterBright WalterBright merged commit 4d3bbfb into dlang:master Mar 23, 2014
@9rnsr 9rnsr deleted the fix12378 branch March 29, 2014 07:11
WalterBright pushed a commit that referenced this pull request Sep 6, 2014
AST identity issue has been properly fixed in #3383 (fceb16b).
@CyberShadow
Copy link
Member

This pull request may have introduced a regression:
https://issues.dlang.org/show_bug.cgi?id=14739

@CyberShadow
Copy link
Member

This pull request introduced a regression:
https://issues.dlang.org/show_bug.cgi?id=15092

@CyberShadow
Copy link
Member

This pull request introduced a regression:
https://issues.dlang.org/show_bug.cgi?id=17492

@ibuclaw
Copy link
Member

ibuclaw commented Jul 1, 2018

I would say this pull request introduced a regression, but it may have already be reported.

fail_compilation/fail2789.d(103): Error: function `fail2789.mul14147(const(int[]) left, const(int[]) right)` conflicts with previous declaration at fail_compilation/fail2789.d(99)
---
*/
struct S14147(alias func)
{
}
pure auto mul14147(const int[] left, const int[] right)
{
S14147!(a => a) s;
}
pure auto mul14147(const int[] left, const int[] right)
{
S14147!(a => a) s;
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants