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 19891 - Confusing error messages for auto ref parameters with default values #9894
Conversation
I think so. |
No. It's a minor issue, and not a regression. |
All bug fixes should target stable unless they are big/complex/introduce breaking changes. |
@@ -4154,6 +4154,7 @@ extern (C++) final class TypeFunction : TypeNext | |||
ubyte iswild; // bit0: inout on params, bit1: inout on qualifier | |||
Expressions* fargs; // function arguments | |||
int inuse; | |||
bool incomplete; // return type or default arguments removed |
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.
I don't really get what this means.
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.
Wait until I mark this PR ready for review, I found an issue which may result in removing that flag.
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.
What do you suggest would be more descriptive.
When "All" is used, one should be careful not to drive over a cliff. In this case, it's a trivial refinement, and not worth the risk of destabilizing the stable version. |
It's not my rules. |
d65ba9f
to
b8869f6
Compare
…th default values
If no argument is given then then the default argument decides `ref`, and if an argument is present then the default argument is never used.
Thanks for your pull request and interest in making D better, @SSoulaimane! We are looking forward to reviewing it, and you should be hearing from a maintainer soon.
Please see CONTRIBUTING.md for more information. If you have addressed all reviews or aren't sure how to proceed, don't hesitate to ping us with a simple comment. 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#9894" |
19891.
Should this go stable?