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
Remove internals for typedef declaration #4038
Conversation
Also needs druntime tweaking: dlang/druntime#978 |
Looks like druntime and phobos are ready for this now. Time to merge? |
Yes, ready to merge. |
I'm ready for this. Any last objections @WalterBright, @andralex? |
Was the |
Surely immediate error was not good. I made the |
Let me take a look at this after I get home. Thanks! |
@WalterBright: When would that be, approximately? The PR should be good to go, although the |
if (targ->ty != Ttypedef) | ||
goto Lno; | ||
tded = ((TypeTypedef *)targ)->sym->basetype; | ||
goto Lno; | ||
break; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The break;
has become dead code.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right, fixed.
Auto-merge toggled on |
Remove internals for typedef declaration
Supplemental PR for the change: #4029
This PR also make the syntax
is(T == typedef)
an error. To do it, this PR needs corresponding Phobos fix:dlang/phobos#2577