Skip to content
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

dmd.expression: Remove Expression.op_overload member function #8798

Merged
merged 1 commit into from Oct 5, 2018

Conversation

ibuclaw
Copy link
Member

@ibuclaw ibuclaw commented Oct 4, 2018

Front-end only function, and it only forwards to the free function in opover.d.

Might as well remove it and let UFCS take care of the rest.

@ibuclaw ibuclaw added the Refactoring No semantic changes to code label Oct 4, 2018
@dlang-bot
Copy link
Contributor

Thanks for your pull request, @ibuclaw!

Bugzilla references

Your PR doesn't reference any Bugzilla issue.

If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog.

Testing this PR locally

If 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#8798"

@Geod24 Geod24 closed this Oct 4, 2018
@Geod24 Geod24 reopened this Oct 4, 2018
@thewilsonator
Copy link
Contributor

Semaphore timeout again.

@Geod24 Geod24 closed this Oct 5, 2018
@Geod24 Geod24 reopened this Oct 5, 2018
@dlang-bot dlang-bot merged commit 9f8f054 into dlang:master Oct 5, 2018
@ibuclaw ibuclaw deleted the rmopoverload branch October 5, 2018 06:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge Refactoring No semantic changes to code
Projects
None yet
4 participants