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

Another fix for Issue 3144 - Invalid break accepted #1182

Merged
merged 1 commit into from Oct 27, 2012

Conversation

9rnsr
Copy link
Contributor

@9rnsr 9rnsr commented Oct 14, 2012

http://d.puremagic.com/issues/show_bug.cgi?id=3144

This is another pull request for issue 3114, which against #660.
This doesn't reject switch(cond) with(exp) { ... } idiom.

@WalterBright
Copy link
Member

Does this replace #660?

@nazriel
Copy link
Contributor

nazriel commented Oct 22, 2012

@WalterBright, yes.
@yebblies PR didn't take switch (foo) with (Foo) into account while this one does

yebblies added a commit that referenced this pull request Oct 27, 2012
Another fix for Issue 3144 - Invalid break accepted
@yebblies yebblies merged commit 86a378b into dlang:master Oct 27, 2012
@yebblies
Copy link
Member

@9rnsr This issue is likely caused by the same part of the parser, it might be worth looking into.

@9rnsr
Copy link
Contributor Author

9rnsr commented Oct 27, 2012

For the bug 4283, I've opened new pull request: #1230

yebblies added a commit that referenced this pull request Nov 8, 2012
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