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

Fix Issue 20147 - Enable comparison (==, >, >=, <=, <) between std.bigint.BigInt and floating point numbers #7172

Merged
merged 1 commit into from Sep 7, 2019

Conversation

n8sh
Copy link
Member

@n8sh n8sh commented Sep 6, 2019

Split off as its own PR separate from #7151 because there is a single mathematically correct way to compare numbers without needing to consider rounding mode.

@dlang-bot
Copy link
Contributor

dlang-bot commented Sep 6, 2019

Thanks for your pull request, @n8sh!

Bugzilla references

Auto-close Bugzilla Severity Description
20147 enhancement Enable comparison (==, >, >=, <=, <) between std.bigint.BigInt and floating point numbers

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 + phobos#7172"

@n8sh n8sh force-pushed the issue-20147 branch 3 times, most recently from ba77195 to ebf7fad Compare September 6, 2019 02:34
@n8sh n8sh force-pushed the issue-20147 branch 6 times, most recently from 9a01dc9 to e303902 Compare September 6, 2019 05:41
@n8sh n8sh force-pushed the issue-20147 branch 2 times, most recently from 3d56086 to ce3e7f6 Compare September 7, 2019 00:40
@thewilsonator
Copy link
Contributor

Good to go?

@n8sh
Copy link
Member Author

n8sh commented Sep 7, 2019

Yep

@dlang-bot dlang-bot merged commit d3e9671 into dlang:master Sep 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants