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 17139 - dscanner needs to handle 'scope' function attributes #5085
Conversation
|
I would disable the |
This is another thing that's not handled by libdparse. I see exactly what's the problem but it's not fixable quickly, it requires @Hackerpilot intervention. This is like dlang-community/libdparse#106. There's 3rd rule that's still not handled. |
std/json.d
Outdated
@@ -37,7 +37,7 @@ import std.traits; | |||
|
|||
// check a type | |||
long x; | |||
if (const(JSONValue)* code = "code" in j) | |||
if (auto code = "code" in j) |
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 you use const here instead of auto?
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.
const identifier now works but it breaks const(Type). because it predates the typectors which are also used for the rule I broke. I'm gonna make a libdparse PR.
@JackStouffer, sorry for the mishaps but this is good now. |
Auto-merge toggled on |
No description provided.