-
-
Notifications
You must be signed in to change notification settings - Fork 606
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 20742 - dmd -X (JSON output) includes uncompiled symbols #11044
Conversation
PR dlang#6621 took the code for pending `ConditionalDeclaration`s from doc.d but missed the early return.
Thanks for your pull request and interest in making D better, @MoonlightSentinel! 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 run digger -- build "stable + dmd#11044" |
Thanks! I will see if I can test this against the project I found it in to see if it fixes the issue. |
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 was able to checkout this PR and verify that it fixes the original problem I was having. Thanks!
I'm reading the code, and I'm not sure how this works. What does I'm always interested in learning more about how the compiler works, but I never have been able to approach it... |
According to the documentation Lines 43 to 48 in e7b6012
|
OK, that makes sense. Edit: Ah, nevermind. I see that it's checking for "notComputed", not "notIncluded". 🤦 |
Anyone with more experience who can review this? It seems simple, and it would be nice to get it into 2.091.1 |
Too late for this :( |
Forget waiting, this seems so simple, I'm just going to merge it. It's only the json output anyway. |
PR #6621 took the code for pending
ConditionalDeclaration
s fromdoc.d but missed the early return.
CC @schveiguy
Reference:
dmd/src/dmd/doc.d
Lines 1202 to 1219 in 83a929e