Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Prevent comparison and dereferencing of raw pointers in constexprs #25840

Merged
merged 1 commit into from
May 28, 2015

Conversation

arielb1
Copy link
Contributor

@arielb1 arielb1 commented May 27, 2015

Fixes #25826.

@rust-highfive
Copy link
Collaborator

r? @alexcrichton

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

@arielb1
Copy link
Contributor Author

arielb1 commented May 27, 2015

r? @nrc

Prohibiting equality (which is needed for #25768) hurts inference too much - it can wait for rust-lang/rfcs#1135.

@rust-highfive rust-highfive assigned nrc and unassigned alexcrichton May 27, 2015
// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
// option. This file may not be copied, modified, or distributed
// except according to those terms.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you add a comment saying what is being tested

@nrc
Copy link
Member

nrc commented May 27, 2015

r=me with the comment added

@arielb1 arielb1 force-pushed the ptr-compare-fixes branch 4 times, most recently from 9fe22e6 to 7d69a71 Compare May 28, 2015 00:03
@nrc
Copy link
Member

nrc commented May 28, 2015

@bors r+

@bors
Copy link
Contributor

bors commented May 28, 2015

📌 Commit 7d69a71 has been approved by nrc

@arielb1 arielb1 force-pushed the ptr-compare-fixes branch from 7d69a71 to 080311d Compare May 28, 2015 00:23
@nrc
Copy link
Member

nrc commented May 28, 2015

@bors r+

@bors
Copy link
Contributor

bors commented May 28, 2015

📌 Commit 080311d has been approved by nrc

bors added a commit that referenced this pull request May 28, 2015
@bors
Copy link
Contributor

bors commented May 28, 2015

⌛ Testing commit 080311d with merge 6a3d55a...

@bors bors merged commit 080311d into rust-lang:master May 28, 2015
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Raw pointer comparisons are valid constant expressions
5 participants