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
fix Issue 16607 - forward reference error for nested struct #6214
Conversation
- This reverts removing the AggregateDeclaration.setScope override (see 75b5b69) and fixes a wrong error triggered by setting _scope on a SIZEOKdone struct during deferred setScope phases (outer struct has forward references).
|
@@ -0,0 +1,15 @@ | |||
struct A(T) | |||
{ | |||
T t; // causes A to be SIZEOKfwd b/c B (passed as T) isn't yet done |
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.
Thank you! But would it be much to ask to avoid shortening words in commits in the future? They might not be very accessible to everyone depending on their English level.
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.
Mmh, only unwillingly, I'm fairly used to that style and have a lot of stuff to type each day.
I wouldn't do that in a changelog entry or specs, but I don't see how that increases any hurdle.
If you're already starring at test16607.d, you're deep into mud anyhow.
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.
Sorry, never mind.
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.
Thanks for raising this, getting understood is important, and this is a useful consideration.
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.
Looks good to me.
75b5b69) and fixes a wrong error triggered by setting _scope on a
SIZEOKdone struct during deferred setScope phases (outer struct has
forward references).