-
-
Notifications
You must be signed in to change notification settings - Fork 598
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
Remove unused FLAGS aliases #7752
Conversation
Thanks for your pull request, @wilzbach! Bugzilla referencesYour PR doesn't reference any Bugzilla issue. If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog. |
src/dmd/lexer.d
Outdated
@@ -1988,9 +1983,10 @@ class Lexer | |||
if (base == 8 && n >= 8) | |||
error("octal literals `0%llo%.*s` are no longer supported, use `std.conv.octal!%llo%.*s` instead", n, p - psuffix, psuffix, n, p - psuffix, psuffix); | |||
TOK result; | |||
with(FLAGS) |
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'd rather not start using with
.
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.
No with
0cad4df
to
e7ea8c1
Compare
e7ea8c1
to
046b2a0
Compare
(removed `with) |
Requested changes were made.
Argh Jenkins seems to be losing PRs more and more frequently :/ |
They are only used once below. We can even use
with
to make it look nicer.