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

Create a FreshFloatTy separate from FreshIntTy #25344

Merged
merged 1 commit into from
May 13, 2015

Conversation

arielb1
Copy link
Contributor

@arielb1 arielb1 commented May 12, 2015

There is no subtyping relationship between the types (or their non-freshened
variants), so they can not be merged.

Fixes #22645
Fixes #24352
Fixes #23825

Fixes #25235 (no test in issue).
Fixes #19976 (test is outdated).
Fixes #18623.
Fixes #18954
Fixes #23722
Fixes #24138

There is no subtyping relationship between the types (or their non-freshened
variants), so they can not be merged.

Fixes rust-lang#22645
Fixes rust-lang#24352
Fixes rust-lang#23825

Should fix rust-lang#25235 (no test in issue).
Should fix rust-lang#19976 (test is outdated).
@rust-highfive
Copy link
Collaborator

r? @pcwalton

(rust_highfive has picked a reviewer for you, use r? to override)

@arielb1
Copy link
Contributor Author

arielb1 commented May 12, 2015

r? @nikomatsakis

@arielb1
Copy link
Contributor Author

arielb1 commented May 12, 2015

More duplicates:

#18623
#18954
#23722
#24138
#25036 (I think)

@nikomatsakis
Copy link
Contributor

@bors r+

@bors
Copy link
Contributor

bors commented May 12, 2015

📌 Commit 36eb09f has been approved by nikomatsakis

bors added a commit that referenced this pull request May 13, 2015
There is no subtyping relationship between the types (or their non-freshened
variants), so they can not be merged.

Fixes #22645
Fixes #24352
Fixes #23825

Should fix #25235 (no test in issue).
Should fix #19976 (test is outdated).
@bors
Copy link
Contributor

bors commented May 13, 2015

⌛ Testing commit 36eb09f with merge 222cd73...

@bors bors merged commit 36eb09f into rust-lang:master May 13, 2015
@shepmaster
Copy link
Member

@arielb1 That is an impressive number of issues fixed in one fell swoop! Very nice 👏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment