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 18708 - Flow analysis in constructors not done correctly fo… #8118
Conversation
Thanks for your pull request, @WalterBright! Bugzilla references
Testing this PR locallyIf you don't have a local development environment setup, you can use Digger to test this PR: dub fetch digger
dub run digger -- build "master + dmd#8118" |
190e73f
to
e666061
Compare
src/dmd/ctorflow.d
Outdated
/****************************** | ||
* OR CSX bits to `this` | ||
* Params: | ||
* from = bits to OR in |
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 name of the parameter is ctorflow
.
@@ -0,0 +1,64 @@ | |||
/* TEST_OUTPUT: | |||
--- | |||
fail_compilation/test18708.d(24): Error: one path skips field `s` |
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.
Can the missing path be included in the error message?
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.
Changing error messages is beyond the scope of this PR.
ctorflow.freeFieldinit(); | ||
} | ||
} | ||
enclosing.ctorflow.OR(ctorflow); |
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.
Indentention looks inconsistent.
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.
It isn't. This line is in an if block.
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.
Ok, I’m a bit limited here on my phone.
…r if, ||, &&