Skip to content

Conversation

varkor
Copy link
Contributor

@varkor varkor commented Oct 21, 2019

It already has the right form, so this is just a renaming. Fixes #65655.

r? @eddyb

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 21, 2019
@varkor varkor added the F-const_generics `#![feature(const_generics)]` label Oct 21, 2019
@eddyb
Copy link
Member

eddyb commented Oct 21, 2019

r=me modulo nits

@rust-highfive

This comment has been minimized.

@varkor varkor force-pushed the canonical-const-to-bound-const branch from 2cb286c to 6147784 Compare October 21, 2019 15:46
@varkor
Copy link
Contributor Author

varkor commented Oct 21, 2019

@bors r=eddyb rollup

@bors
Copy link
Collaborator

bors commented Oct 21, 2019

📌 Commit 6147784eb5e72d25df95213f2d8a34f02f47514a has been approved by eddyb

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 21, 2019
@bors

This comment has been minimized.

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Oct 21, 2019
@varkor varkor force-pushed the canonical-const-to-bound-const branch from 6147784 to e9c2685 Compare October 21, 2019 19:50
@varkor
Copy link
Contributor Author

varkor commented Oct 21, 2019

@bors r=eddyb rollup

@bors
Copy link
Collaborator

bors commented Oct 21, 2019

📌 Commit e9c2685 has been approved by eddyb

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Oct 21, 2019
Centril added a commit to Centril/rust that referenced this pull request Oct 21, 2019
…st, r=eddyb

Rename `ConstValue::Infer(InferConst::Canonical(..))` to `ConstValue::Bound(..)`

It already has the right form, so this is just a renaming. Fixes rust-lang#65655.

r? @eddyb
bors added a commit that referenced this pull request Oct 22, 2019
Rollup of 7 pull requests

Successful merges:

 - #62330 (Change untagged_unions to not allow union fields with drop)
 - #65092 (make is_power_of_two a const function)
 - #65621 (miri: add write_bytes method to Memory doing bounds-checks and supporting iterators)
 - #65647 (Remove unnecessary trait bounds and derivations)
 - #65653 (keep the root dir clean from debugging)
 - #65660 (Rename `ConstValue::Infer(InferConst::Canonical(..))` to `ConstValue::Bound(..)`)
 - #65663 (Fix typo from #65214)

Failed merges:

r? @ghost
@bors bors merged commit e9c2685 into rust-lang:master Oct 22, 2019
@varkor varkor deleted the canonical-const-to-bound-const branch January 2, 2020 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
F-const_generics `#![feature(const_generics)]` S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Replace ConstValue::Infer(InferConst::Canonical(..)) with ConstValue::Bound
4 participants